Skip to content
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

fix setuptools version #381

Merged
merged 1 commit into from
Feb 27, 2025
Merged

fix setuptools version #381

merged 1 commit into from
Feb 27, 2025

Conversation

lsf37
Copy link
Member

@lsf37 lsf37 commented Feb 27, 2025

sel4-deps have started to fail installation on GitHub CI, although they seem to work fine on clean Ubuntu 22.04.

See also the saga on pypa/setuptools#4483 -- fixing the version of setuptools seems to fix the problem for CI.

sel4-deps have started to fail installation on GitHub CI, although they
seem to work fine on clean Ubuntu 22.04.

See the saga on pypa/setuptools#4483

Signed-off-by: Gerwin Klein <[email protected]>
@lsf37
Copy link
Member Author

lsf37 commented Feb 27, 2025

Have tested this with the style check on this PR (commit now dropped) and it passed. I have not consistently see it fail everywhere, though, so it's unclear if it will really fix the problem or just not make it worse.

@lsf37 lsf37 marked this pull request as ready for review February 27, 2025 01:39
@lsf37 lsf37 requested a review from corlewis February 27, 2025 01:39
@lsf37
Copy link
Member Author

lsf37 commented Feb 27, 2025

At least it is now showing the failure here, so it did definitely fix something when I tested it.

@lsf37 lsf37 changed the title fixe setuptools version fix setuptools version Feb 27, 2025
@lsf37 lsf37 added the dependencies Pull requests that update a dependency file label Feb 27, 2025
@lsf37 lsf37 merged commit 6545ead into master Feb 27, 2025
1 of 7 checks passed
@lsf37 lsf37 deleted the setup-tools branch February 27, 2025 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants