-
Notifications
You must be signed in to change notification settings - Fork 130
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
[ci] Update to GCC12 #940
[ci] Update to GCC12 #940
Conversation
6847270
to
73cbe24
Compare
All CI issues should be fixed now, including the broken Mac OS build (if they merge the PR and avr-gcc does not fail because of |
Thank you so much for fixing this, I fell asleep frustrated. |
2b43703
to
3026038
Compare
Ok, avr-gcc 12 has been merged on macOS and I managed to make the new compile flags backward compatible with GCC10 and GCC11. Ran on as many examples in hardware as I was motivated, didn't find anything wrong. Should be good to go! |
Hm, the docs.modm.io generator script also uses multiprocessing and the Python 3.9 bug causes the job to hang. Have they not tested this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks.
fd47e53
to
9b6beaa
Compare
f4e5d35
to
e6e0cc5
Compare
I think there is an issue with the Ubuntu 22.04 update, rather than just Python multiprocessing code. I think specifically in the deduplication effort of the documentation generator, so I suspect something changed with the filesystem between 20.04 and 22.04. |
RAM space is running out for some tests.
co-authored-by: Christopher Durand <christopher.durand@rwth-aachen.de>
Processes are now spawned instead of forked, which could otherwise cause lockups and hung processes.
co-authored-by: Christopher Durand <christopher.durand@rwth-aachen.de>
I've changed the container back to the previous one only for the build-docs-test. It doesn't use GCC anyways, only Python3 and I'm not motivated to debug the multiprocessing issues further. |
Updating modm to GCC12 to keep up with the times.
examples_compile.py
and test all script