-
Notifications
You must be signed in to change notification settings - Fork 57
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
Release v0.13 #250
Release v0.13 #250
Conversation
3109531
to
1eb3c3d
Compare
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.
LGTM. Ran some adapts and all looked fine.
export FC=gfortran-12 | ||
export LDFLAGS='-fuse-ld=lld' | ||
elif [[ "${{ matrix.compiler }}" == 'gcc' ]]; then | ||
export CC=gcc-13 | ||
export CXX=g++-13 | ||
export FC=gfortran-13 | ||
export CC=gcc-12 | ||
export CXX=g++-12 | ||
export FC=gfortran-12 |
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.
Is there an issue with gcc13?
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.
No issue, it just got removed from the Ubuntu image used for GitHub Actions (not sure why). See actions/runner-images@cfcc1c6.
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.
I guess this was an intentional decision: actions/runner-images#9866.
c0ef2fe
to
e2ab101
Compare
e2ab101
to
2a7f215
Compare
Final changes for Palace version 0.13.