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

Pin pip<23.1 and setuptools<67 to allow PEP-440 non compliance #653

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

ajkavanagh
Copy link
Contributor

@ajkavanagh ajkavanagh commented Mar 9, 2023

PEP-440 is more strict about requirement lines. As an example:

pytz>dev

is non-compliant, but is in 5.2.4 of kombu's requirements. This breaks building wheelhouses from source (such as the octavia charm). This patch pins pip and setuptools to the latest versions that will defintely still allow PEP-440 non-compliant packages to install.

The option --upgrade-buildvenv-core-deps can be used to override this and will install the latest versions of pip and setuptools available.

Fixes-Bug: #652

Description of change

Checklist

  • Are all your commits [logically] grouped and squashed appropriately?
  • Does this patch have code coverage?
  • Does your code pass make test?

PEP-440 is more strict about requirement lines.  As an example:

    pytz>dev

is non-compliant, but is in 5.2.4 of kombu's requirements.  This breaks
building wheelhouses from source (such as the octavia charm).  This
patch pins pip and setuptools to the latest versions that will defintely
still allow PEP-440 non-compliant packages to install.

The option --upgrade-buildvenv-core-deps can be used to override this
and will install the latest versions of pip and setuptools available.

Fixes-Bug: juju#652
@ajkavanagh ajkavanagh requested a review from fnordahl March 9, 2023 17:29
@ajkavanagh
Copy link
Contributor Author

The integration test failure is unrelated to the change; sadly it looks like a networking issue within charmcraft.

@javacruft javacruft merged commit 1847cbe into juju:master Mar 14, 2023
@ajkavanagh ajkavanagh deleted the pin-for-pep440 branch March 14, 2023 20:50
ajkavanagh added a commit to ajkavanagh/charm-tools that referenced this pull request Mar 15, 2023
to allow PEP-440 non compliance.

(backported relevant parts from master branch in juju#653)

PEP-440 is more strict about requirement lines.  As an example:

    pytz>dev

is non-compliant, but is in 5.2.4 of kombu's requirements.  This breaks
building wheelhouses from source (such as the octavia charm).  This
patch pins pip and setuptools to the latest versions that will defintely
still allow PEP-440 non-compliant packages to install.

The option --upgrade-buildvenv-core-deps can be used to override this
and will install the latest versions of pip and setuptools available.

Fixes-Bug: juju#652
ajkavanagh added a commit to ajkavanagh/charm-tools that referenced this pull request Mar 15, 2023
to allow PEP-440 non compliance.

(backported relevant parts from master branch in juju#653)

PEP-440 is more strict about requirement lines.  As an example:

    pytz>dev

is non-compliant, but is in 5.2.4 of kombu's requirements.  This breaks
building wheelhouses from source (such as the octavia charm).  This
patch pins pip and setuptools to the latest versions that will defintely
still allow PEP-440 non-compliant packages to install.

The option --upgrade-buildvenv-core-deps can be used to override this
and will install the latest versions of pip and setuptools available.

Fixes-Bug: juju#652
javacruft pushed a commit that referenced this pull request Mar 15, 2023
to allow PEP-440 non compliance.

(backported relevant parts from master branch in #653)

PEP-440 is more strict about requirement lines.  As an example:

    pytz>dev

is non-compliant, but is in 5.2.4 of kombu's requirements.  This breaks
building wheelhouses from source (such as the octavia charm).  This
patch pins pip and setuptools to the latest versions that will defintely
still allow PEP-440 non-compliant packages to install.

The option --upgrade-buildvenv-core-deps can be used to override this
and will install the latest versions of pip and setuptools available.

Fixes-Bug: #652
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.

2 participants