Time  Nick        Message
07:31 ROllerozxa  celeron55: #13675 requires some involvement from you for DNS and repository settings stuff
07:31 ShadowBot   https://github.com/minetest/minetest/issues/13675 -- Build MkDocs Lua API docs using GitHub CI, deploy to api.minetest.net by rollerozxa
10:02 Sokomine    curent git version of mt doesn't compile for me? mostly get complaints about video::EMF_*
10:03 ROllerozxa  irrlichtmt master currently isn't compatible with minetest master, you need to check out the 1.9.0mt11 tag to compile
10:04 Sokomine    aah! thank you
10:12 nrz_        sfan5, time to merge those repos ! 🙂
10:12 sfan5       no
10:12 nrz_        i know i didn't finished my cleanup MR, i was on holidays, i'll try to take some time soon
10:12 sfan5       time to use a submodule
10:12 nrz_        NO au secours help kill me
10:13 nrz_        submodule hell will happen, am i the only one to have done development in software industry with this to say it's cancer ?
10:13 nrz_        i prepare popcorn for people missing git submodule update --init on each git pull :p
10:13 nrz_        (and opening issues obviously)
10:14 ROllerozxa  yeah instead of now when people open issues because their naively cloned from master irrlichtmt not compiling with minetest
10:14 nrz_        and people publishing submodule update by error in PR because they miss update it before commiting and after that taking dozens of minutes to fix because it's hell to fix the submodule commit after another commit haha
10:15 nrz_        submodule is good if this lib is re-usable by another project, maybe, (i prefer a standalone build and just download the lib), but who will re-use it ? no one 🙂
10:46 Sokomine    *careful question* how do i checko ut the tag 1.9.0mt11? didn't get git to do it. sure, ought to know. but someone else has the same problem, they'll at least find help here in the log i hope
10:46 Sokomine    submodules were difficult with the dreambuilder modpack iirc
10:49 nrz_        i extracted one commit from my previous PR in order to rework this MR to be closer to the initial objective, i'll try to iterate on multiple PR, preventing too big PR
10:49 nrz_        here is the pragma one (+ some gitignore additions): https://github.com/minetest/irrlicht/pull/224, easy to review & validate
10:55 nrz_        that'll permit to reduce drastically the other PR and i'll try to split the problematic differently/iteratively
12:07 nrz_        sfan5: we cannot blacklist all, see: https://github.com/minetest/irrlicht/tree/master/cmake
12:08 nrz_        ah no err 😄 yes we can,
12:08 nrz_        my bad
12:08 nrz_        it's amended
15:04 nrz_        ty sfan5
17:26 disba1ancer maybe it's better to use fetchcontent instead of submodules?
19:44 sfan5       planning to merge #13622 within 20m
19:44 ShadowBot   https://github.com/minetest/minetest/issues/13622 -- Refactor parts of Irrlicht's material system by grorp
19:45 sfan5       (tagging irrlichtmt was taken care of so the build will work)
19:51 grorp       yay!