Time  Nick         Message
10:13 sfan5        pushing http://sprunge.us/wV1MMf?diff in 5m
10:34 erlehmann    sfan5 do you not make a PR for this?
10:34 sfan5        no, why would I?
10:36 erlehmann    ah, i forgot minetest has no mandatory reviews
10:36 erlehmann    i mean, only for feature stuff in main repo, right?
10:36 erlehmann    not for typo
10:36 sfan5        there is reviews for everything except for small bugfixes
10:37 erlehmann    btw if you tag something low priority, does it mean “too much effort to put much work into to fix it” or “too unimportant to put much work into to fix it”
10:37 erlehmann    ?
10:38 erlehmann    bc i think depending on issue it could be either
10:39 sfan5        or just "it's not important if this isn't fixed soon"
10:53 erlehmann    ok
13:51 erlehmann    <Fleckenstein> the entire irrlicht situation would be better if irrlichtmt was included as a submodule
13:51 erlehmann    <Fleckenstein> this way, when doing a bisect on the minetest repo, the irrlicht commit that suits that particular minetest commit would be selected automatically
13:51 erlehmann    i have suggested this before
13:51 erlehmann    why is it not done?
13:54 dzho         I don't know but I wonder how adept at using submodules people tend to be.
13:58 specing      because minetest is now an engine and you can't have an engine depending on an engine to still be called an engine. Got it?
14:12 rubenwardy   nothing has changed in that regard
14:13 rubenwardy   the real reason is that other devs don't like submodules
14:13 rubenwardy   and they are annoying to deal with
14:28 MTDiscord    <savilli> submodules are annoying indeed
14:29 MTDiscord    <savilli> i wouldn't mind if irrlichtmt was included in main repo as if
14:29 MTDiscord    <savilli> just like lua
14:49 erlehmann    rubenwardy i agree that other devs are annoying to deal with
15:08 MTDiscord    <savilli> nah they're cool
15:09 erlehmann    savilli it was a jokuss
15:09 MTDiscord    <Sublayer plank> lol
15:13 MTDiscord    <Jonathon> is https://github.com/minetest/minetest/issues/11756 something that can be marked with possible close?
15:22 erlehmann    so about submodules, what's annoying about them? i might be able to provide a workaround (think, post-checkout hook)
15:23 MTDiscord    <Jonathon> github zip download doesnt support them btw
15:23 MTDiscord    <josiah_wi> A proposed solution and an offer to implement it! 9/10
15:24 MTDiscord    <josiah_wi> Arguably git zip download is the wrong way to get source code. I'm not sure why that feature even exists.
15:25 MTDiscord    <Jonathon> useful for non tech people
15:26 MTDiscord    <Sublayer plank> yeah
15:26 MTDiscord    <Jonathon> arguably you could say people getting source should know something, but thats only true for complied langauges, something like python/lua downloading and running is the most they can do
15:28 MTDiscord    <josiah_wi> Is it too much to ask for people who want to compile Minetest from source to learn basic git usage? I like your argument for Python/Lua, but Minetest is compiled, and there's the IrrlichtMt fiasco anyway, so its not like downloading the zip even works out of the box rn.
15:29 MTDiscord    <josiah_wi> Absorbing IrrlichtMt into Minetest like Savilli suggested would solve that, of course.
15:29 MTDiscord    <Jonathon> absorbing it now can lead to problems
15:29 MTDiscord    <Jonathon> i think its better to keep it separate while its cut down/rewritten and the remains can be merged into mte
15:30 MTDiscord    <Jonathon> any which way you dice it you will have issues
15:32 erlehmann    well, adding it as a submodule literally doesn't do anything if you don't use submodules
16:25 erlehmann    uh could it be that support for A1G5G5B5 (a.k.a. 1 bit alpha, 5 bits each RGB) is broken?
16:25 erlehmann    maybe a driver issue
16:38 MTDiscord    <josiah_wi> Did git bisect get anywhere yet?
16:38 erlehmann    oh no, my error
16:38 erlehmann    alpha was 0
18:20 Fleckenstein is there a specific reason some people seem to be able to access this channel from MTDiscord but I can't?
18:21 MTDiscord    <Sublayer plank> to speak in #minetest-dev from discord you need to have the regular role or higher
18:22 Fleckenstein hmm ok why tho i think it used to be different?
18:29 MTDiscord    <Jonathon> no idea, always been that way
18:30 MTDiscord    <Jonathon> i believe its this way to proxy mirror irc, not sure if channels have there rules on how you chat (no irc web clients, +v in hub, etc)
22:26 MTDiscord    <exe_virus> I didn't understand git until I worked a job professionally in the industry
22:26 MTDiscord    <exe_virus> And used GitHub zip download religiously
22:54 MTDiscord    <SX> Same, I only understood SVN :p