-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
devcontainer: update to Fedora 40 #118161
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
Conversation
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
37 has been EOL since December last year, 38 is going to be EOL in a month or so, 39 is current stable version, 40 was released today.
86435b3
to
ed7d4ba
Compare
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
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.
Oh, this is great! I was actually about to open this same PR since 1) the whole EOL timeline on Fedora 37/38 + the big 40 announcement, and 2) we need a later version of clang to build with the JIT enabled.
Looks like the PR title doesn't match the version you're updating to (39 vs. 40) but that's a nit :)
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, I will merge this PR after testing in my local environment.
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.
Sorry, I should have retested this earlier but it looks like I'm running into a make
error when I try to update to Fedora 40, on updateContentCommand
in the devcontainer.json
.
Are you not seeing an error?
Specifically, this is failing:
"docs": [
"make",
"--directory",
"Doc",
"venv",
"html"
]
No. Log
|
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.
Okay, I think that my VS Code install is broken. I ended up testing both in a fresh Codespaces instance and a local dev container with VS Code insiders and managed to get everything working as expected. LGTM.
I am merging this PR, thank you for the cross-checking @savannahostrowski |
Fedora 37 has been EOL since December last year, 38 is going to be EOL in May.
Tested it and no further changes seem necessary.