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

Fix pip reinstallation on windows due to constraints #6756

Merged
merged 1 commit into from
May 19, 2022

Conversation

greshilov
Copy link
Contributor

What do these changes do?

Fix for constantly failing CI on windows agents.

Description

We have pinned pip version in constraints.txt file:

So when you call pip install ... -c constraints.txt it will downgrade/upgrade pip depending on the currently installed version. This works fine on all platforms except windows, which forces you to explicitly use python -m pip install ... when dealing with the pip package itself.

Since python -m pip install ... command is a completely legit replacement for pip install ... (examples from the pip wiki), I decided to update Makefile with it.

@greshilov
Copy link
Contributor Author

@Dreamsorcerer PTAL

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #6756 (28226bc) into master (e16fb62) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6756   +/-   ##
=======================================
  Coverage   93.44%   93.44%           
=======================================
  Files         104      104           
  Lines       30627    30627           
  Branches     3076     3076           
=======================================
  Hits        28620    28620           
  Misses       1838     1838           
  Partials      169      169           
Flag Coverage Δ
unit 93.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 08ca779...28226bc. Read the comment docs.

@Dreamsorcerer
Copy link
Member

Weird. Fine by me.

@Dreamsorcerer Dreamsorcerer added the bot:chronographer:skip This PR does not need to include a change note label May 18, 2022
@Dreamsorcerer
Copy link
Member

Hmm, Timeline protection seems to have broken. I guess it's a remote call, so maybe the server is down?

@greshilov
Copy link
Contributor Author

Hmm, Timeline protection seems to have broken. I guess it's a remote call, so maybe the server is down?

@webknjaz should know, I suppose.

@webknjaz
Copy link
Member

Urgh... Must be the same problem as with Patchback.. sanitizers/patchback-github-app#27

@webknjaz
Copy link
Member

Should be fixed now.

@webknjaz webknjaz added bot:chronographer:skip This PR does not need to include a change note and removed bot:chronographer:skip This PR does not need to include a change note labels May 18, 2022
@Dreamsorcerer Dreamsorcerer merged commit 72d3d4b into aio-libs:master May 19, 2022
@patchback
Copy link
Contributor

patchback bot commented May 19, 2022

Backport to 3.9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.9/72d3d4b1f68cca5ad15ef50bffb0419b798c7f23/pr-6756

Backported as #6759

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@patchback
Copy link
Contributor

patchback bot commented Sep 7, 2022

Backport to 3.8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.8/72d3d4b1f68cca5ad15ef50bffb0419b798c7f23/pr-6756

Backported as #6911

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Sep 7, 2022
webknjaz pushed a commit that referenced this pull request Sep 7, 2022
…ue to constraints (#6911)

**This is a backport of PR #6756 as merged into master
(72d3d4b).**

## What do these changes do?

Fix for constantly failing CI on windows agents.

## Description

We have pinned pip version in constraints.txt file:

https://github.com/aio-libs/aiohttp/blob/08ca779e9abdb6d558cdd1647fdca17cfe979711/requirements/constraints.txt#L260

So when you call `pip install ... -c constraints.txt` it will
downgrade/upgrade pip depending on the currently installed version. This
works fine on all platforms except windows, which forces you to
explicitly use `python -m pip install ...` when dealing with the pip
package itself.

Since `python -m pip install ...` command is a completely legit
replacement for `pip install ...` ([examples
](https://pip.pypa.io/en/latest/getting-started/)from the pip wiki), I
decided to update `Makefile` with it.

Co-authored-by: Slava <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:skip This PR does not need to include a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants