Development discussion. Logged to https://ddnet.tw/irclogs/ Connected with DDNet's IRC channel, Matrix room and GitHub repositories — IRC: #ddnet on Quakenet | Matrix: #ddnet-developer:matrix.org GitHub: https://github.com/ddnet
Between 2022-10-08 00:00:00Z and 2022-10-09 00:00:00Z
@heinrich5991 I end up missing the generated files way too obvious. Any chance we can somehow trigger generation at the beginning of compilation at all times?
There is one more thing that should be integrated into emerge imo, it should be actively controlling the total amount of jobs instead of total amount of builds
17:15
Launching 3 make -j12's at the same time isn't exactly the sanest of things
I'm going out for supper, but in the meantime, I want to think about what to do next.
20:45
I want there to be a way where if you, say, scan a folder for images, it will also check its directories if possible. If you have a folder inside your skins folder that isn't accounted for. And the editor's Readd function is kind of useless if you also have folders in mapres.(edited)
Yea and that is a reoccurring pain. And can stay undiscovered during development until the build folder is wiped. I find it quite tricky to then fix those issues months later when cloning the repo on a new machine.
(@heinrich5991)
But across repos. Seen it in ddnet++ and even in F-DDrace once.
20:53
It’s a issue in the code base
20:53
Not on the os
20:54
If you are curious I can try getting my broken state from today to not work again
20:55
Im not sure if I mix different issues but it’s always the generated code
20:59
I got it in my vanilla protocol debug branch today. I probably messed up some c++ basics and includes but the end result was a missing include of generated protocol. I ended up doing random restructuring until it worked. But during that time I kinda wished that the code would be generated and linked as early as possible to avoid such issues but not sure how bad that would be
I read heinrich5991's comments on animations PR and why you shouldn't add more config options, but i think this is too big update to just left it as it is. I already posted in #bugs some negative comments by some good players, maybe you received other complaints too, so please consider adding options to disable them.