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

Yes, this is the branch: https://github.com/Miguel-hrvs/TIC-80/t ... 3ds-docker

It fails converting from .elf to .3dsx. This is the .sh that worked before: https://github.com/Miguel-hrvs/TIC-80/b ... to_3dsx.sh


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 :)