Page 1 of 1
New devkitARM builds treat warnings as errors
Posted: Thu Oct 03, 2024 10:27 am
by Miguelhrvs
Hi. I was updating devkitpro's arm docker container to build tic80 for the 3ds and now the compiler is treating the warnings as errors by default. The old version didn't do that. I tried to disable the warnings as errors in gcc without any luck.
Re: New devkitARM builds treat warnings as errors
Posted: Sat Oct 05, 2024 9:11 pm
by WinterMute
Compilers produce warnings for a reason, they should generally be fixed, not switched off. The warnings that have been upgraded to errors are particular sources of bugs - for example function implementations not matching their prototypes.
Re: New devkitARM builds treat warnings as errors
Posted: Tue Oct 08, 2024 4:18 pm
by Miguelhrvs
I managed to fix the errors, but now I get this when it almost ends compilation:
[100%] Linking CXX executable bin/tic80_n3ds.elf
[3DSX] Building metadata
[3DSX] Building binary
Cannot open input file!
Re: New devkitARM builds treat warnings as errors
Posted: Tue Oct 08, 2024 5:52 pm
by WinterMute
Do you have a branch somewhere with errors fixed. Someone can have a look when we get time.
Re: New devkitARM builds treat warnings as errors
Posted: Wed Oct 09, 2024 12:31 pm
by Miguelhrvs
Re: New devkitARM builds treat warnings as errors
Posted: Wed Oct 09, 2024 1:14 pm
by Miguelhrvs
Fixed, I needed to included the .elf extension to the filename in the conversion script.
Re: New devkitARM builds treat warnings as errors
Posted: Sun Oct 13, 2024 8:12 pm
by WinterMute
Nice job fixing those warnings. I was slightly worried there were too many to fix from a brief glance.
I've made a PR to your branch with some tweaks to use the devkitPro cmake functions to build the .3dsx instead of an external shellscript - see https://github.com/Miguel-hrvs/TIC-80/pull/1 Hope it's useful.
Re: New devkitARM builds treat warnings as errors
Posted: Mon Oct 14, 2024 11:20 am
by Miguelhrvs
Thank you, those tweaks are useful, I just merged them in the branch. Cheers