Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Fix cleaninstall test breakage #4224

Merged
merged 1 commit into from
Dec 1, 2021
Merged

Fix cleaninstall test breakage #4224

merged 1 commit into from
Dec 1, 2021

Conversation

moyapchen
Copy link
Contributor

@moyapchen moyapchen commented Dec 1, 2021

...gonna guess this might have to be done again at some point, but python setup.py develop runs cleanly in a new conda environment when I use this.

(Going to guess it'll involve changing all the == to <= or something. Guessing something changed setuptools under the hood at some point maybe... or the dependencies in the other requirements got upgraded enough where some of the documented undefined behavior about how it resolves versioning that setuptools has is biting us. Figuring that out is gonna take much longer than this brute force so w/e.)

Testing steps
Make a bunch of clean conda envs and finagle with requirements until everything installs cleanly lol.

...gonna guess this might have to be done again at some point, but `python setup.py develop` runs cleanly in a new conda environment when I use this.

(Going to guess it'll involve changing all the `==` to `<=` or something. Guessing something changed `setuptools` under the hood at some point maybe... or the depenencies in the other requirements got upgraded enough where some of the documented undefined behavior about how it resolves versioning that setuptools has is biting us.)
Copy link
Contributor

@stephenroller stephenroller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty moya!!!!

@moyapchen moyapchen merged commit 2c31efc into main Dec 1, 2021
@moyapchen moyapchen deleted the dependencies_more branch December 1, 2021 21:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants