New devkitARM builds treat warnings as errors

support for the ARM toolchain
Post Reply
Miguelhrvs
Posts: 5
Joined: Thu Oct 03, 2024 10:13 am

New devkitARM builds treat warnings as errors

Post by Miguelhrvs » Thu Oct 03, 2024 10:27 am

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
Site Admin
Posts: 1968
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: New devkitARM builds treat warnings as errors

Post by WinterMute » Sat Oct 05, 2024 9:11 pm

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.

Help keep devkitPro toolchains free, Donate today

Personal Blog

Miguelhrvs
Posts: 5
Joined: Thu Oct 03, 2024 10:13 am

Re: New devkitARM builds treat warnings as errors

Post by Miguelhrvs » Tue Oct 08, 2024 4:18 pm

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
Site Admin
Posts: 1968
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: New devkitARM builds treat warnings as errors

Post by WinterMute » Tue Oct 08, 2024 5:52 pm

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

Help keep devkitPro toolchains free, Donate today

Personal Blog

Miguelhrvs
Posts: 5
Joined: Thu Oct 03, 2024 10:13 am

Re: New devkitARM builds treat warnings as errors

Post by Miguelhrvs » Wed Oct 09, 2024 12:31 pm

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
Posts: 5
Joined: Thu Oct 03, 2024 10:13 am

Re: New devkitARM builds treat warnings as errors

Post by Miguelhrvs » Wed Oct 09, 2024 1:14 pm

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


WinterMute
Site Admin
Posts: 1968
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: New devkitARM builds treat warnings as errors

Post by WinterMute » Sun Oct 13, 2024 8:12 pm

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.

Help keep devkitPro toolchains free, Donate today

Personal Blog

Miguelhrvs
Posts: 5
Joined: Thu Oct 03, 2024 10:13 am

Re: New devkitARM builds treat warnings as errors

Post by Miguelhrvs » Mon Oct 14, 2024 11:20 am

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


Post Reply

Who is online

Users browsing this forum: No registered users and 3 guests