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

Install twine only when needed [build wheel] #8099

Merged
merged 1 commit into from Jan 7, 2023

Conversation

misl6
Copy link
Member

@misl6 misl6 commented Jan 7, 2023

Maintainer merge checklist

  • Title is descriptive/clear for inclusion in release notes.
  • Applied a Component: xxx label.
  • Applied the api-deprecation or api-break label.
  • Applied the release-highlight label to be highlighted in release notes.
  • Added to the milestone version it was merged into.
  • Unittests are included in PR.
  • Properly documented, including versionadded, versionchanged as needed.

Fixes our RPi wheels workflow (which is broken since a while), as we're installing twine along with the other pip dependencies and twine requires cryptography, which is currently unavailable on piwheels as requires a rust compiler on their runners.

Considering we're also installing twine at the appropriate time (see below), there's no need to keep a duplicated installation.

python3 -m pip install twine

python -m pip install twine

@misl6 misl6 added the Component: tests/CI Tests, CI, GitHub settings label Jan 7, 2023
@misl6 misl6 added this to the 2.2.0 milestone Jan 7, 2023
@misl6 misl6 merged commit b30f798 into kivy:master Jan 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: tests/CI Tests, CI, GitHub settings
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant