-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GCC 13.2 #14
base: master
Are you sure you want to change the base?
Conversation
Merge branch 'bebbo:master' into master
Fetch upstream
Fetch upstream
Fetch upstream
Fetch upstream
Getting a link error with the latest docker on DevilutionX: https://github.com/diasurgical/devilutionX/actions/runs/11059280133/job/30727221901?pr=7435
|
Bugs in the actual toolchain should still be reported in https://github.com/bebbo/amiga-gcc this repo is just for building the docker image. |
gcc13.1 is still highly experimental in the Amiga target. Most amiga devs still use the gcc 6.5 toolchain because of this, just so you're informed of this. @glebm |
@glebm @bebbo found the commit that broke it: bebbo/libnix@f6846ba#diff-a5bd985394076c19822bcb2d40a1618ddcb926607677458f02284ee62d4157d5 /opt/m68k-amigaos/lib/gcc/m68k-amigaos/13.1.1/../../../../m68k-amigaos/bin/ld:
ctype_configure_char.o:(.text._ZNSt5ctypeIcEC1EPKcbm+0x78): undefined reference to `_ctype_' |
@glebm @bebbo i've created an issue here: bebbo/gcc#237 |
@glebm building new image now |
@glebm new image should be available now |
Re-ran the job just now but it's still failing with the same error: https://github.com/diasurgical/devilutionX/actions/runs/11088413258/job/31050491505 |
@glebm might need to switch it up to bebbo's gcc 13.2 branch |
@glebm new image based on 13.2 pushed, let me know if that made it better or worse! |
The build is now green! https://github.com/diasurgical/devilutionX/actions/runs/11206330368/job/31253606044 |
@glebm nice to hear! |
No description provided.