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

Also update packaging python package by default #255

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MaEtUgR
Copy link

@MaEtUgR MaEtUgR commented Nov 21, 2024

When updating setuptools without packaging and installing certain versions of python packages there can be errors because of incompatibility.

For reference I had to juggle around with this exact issue because our CI broke here:
PX4/PX4-Autopilot#23991

I thought I'd rather upstream the solution than keep my local workaround.

When updating setuptools without packaging and installing certain versions of python packages there can be errors because of incompatibility.
@MaEtUgR
Copy link
Author

MaEtUgR commented Nov 21, 2024

I made a test pr to test this in action: PX4/PX4-Autopilot#24016

@MaEtUgR
Copy link
Author

MaEtUgR commented Nov 21, 2024

Works as expected, the installation of empy was failing because of incompatibility without updating packaging as well.
See https://github.com/PX4/PX4-Autopilot/actions/runs/11958904680/job/33339477597?pr=24016
grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant