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

Loosen restriction on packaging version for package compatibility #442

Merged
merged 2 commits into from
Jul 14, 2023

Conversation

bolasim
Copy link
Collaborator

@bolasim bolasim commented Jul 14, 2023

Getting package resolution errors of this nature when trying to install truss.

➜ poetry add truss==0.4.10rc9
Updating dependencies
Resolving dependencies... (0.5s)

Because truss (0.4.10rc9) depends on packaging (>=20.9,<21.0)
 and copier (8.1.0) depends on packaging (>=23.0), truss (0.4.10rc9) is incompatible with copier (8.1.0).
And because no versions of copier match >8.1.0,<9.0.0, truss (0.4.10rc9) is incompatible with copier (>=8.1.0,<9.0.0).
So, because truss-smoke-tester depends on both copier (^8.1.0) and truss (0.4.10rc9), version solving failed.

There are a lot of python packaging and versioning philosophies out there, but one of them is that it's best practice to not put an upper bound unless there is a good reason for it for package incompatibility. Let's stick with that since our package manager poetry is pretty bad at resolving these constraints .

@bolasim bolasim merged commit f52cd38 into main Jul 14, 2023
3 checks passed
@bolasim bolasim deleted the bola/loosen-package branch July 14, 2023 20:52
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.

None yet

2 participants