New devkitARM builds treat warnings as errors

[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.

[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.

[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!

[WinterMute]

Do you have a branch somewhere with errors fixed. Someone can have a look when we get time.

[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

[Miguelhrvs]

Fixed, I needed to included the .elf extension to the filename in the conversion script.

[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.

[Miguelhrvs]

Thank you, those tweaks are useful, I just merged them in the branch. Cheers