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

CI: specify pip < 21.0; avoid using pip3 script #2516

Merged
merged 1 commit into from Feb 7, 2021

Conversation

mwtoews
Copy link
Member

@mwtoews mwtoews commented Feb 7, 2021

Specify pip version less than 21.0 to avoid syntax error with Python 3.5 or older. This fixes CI errors since January 2021.

See this answer for details.

Furthermore, use safer python3 -m pip rather than pip3.

@mwtoews mwtoews merged commit af7cbf0 into OSGeo:master Feb 7, 2021
@kbevers
Copy link
Member

kbevers commented Feb 7, 2021

What are the CI errors? It is probably more sustainable if those can be fixed instead

@mwtoews
Copy link
Member Author

mwtoews commented Feb 7, 2021

The CI errors were attempting to update pip beyond version 21.0 for Python 3.5 or older, which is used for Ubuntu xenial. Upgrading some of these VMs to at least bionic could help.

@mwtoews mwtoews deleted the pin-pip-version branch March 4, 2022 09:53
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