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

Make sure that pip always gets upgraded #211

Merged
merged 2 commits into from
Oct 15, 2020
Merged

Conversation

stweil
Copy link
Collaborator

@stweil stweil commented Oct 14, 2020

It was not upgraded for an existing venv.

Signed-off-by: Stefan Weil sw@weilnetz.de

It was not upgraded for an existing venv.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Copy link
Collaborator

@bertsky bertsky left a comment

Choose a reason for hiding this comment

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

Why not. But I wonder what your use-case is. We never needed to update from the old 9-something default until recently. Does this have to do with the new dependency resolver that will arrive by default soon?

@stweil
Copy link
Collaborator Author

stweil commented Oct 14, 2020

We never needed to update from the old 9-something default until recently.

Maybe most builds did not use an existing venv but created the venv? The Makefile already upgraded pip when it created a new venv.

@bertsky
Copy link
Collaborator

bertsky commented Oct 14, 2020

We never needed to update from the old 9-something default until recently.

Maybe most builds did not use an existing venv but created the venv? The Makefile already upgraded pip when it created a new venv.

No, creating a venv does not update pip automatically. We only noticed this when the old default version (it's 9.0.1 under Ubuntu 18.04) both pre-built wheels in the manylinux2014 format and fallback installation from source distributions using pyproject.toml stopped working.

@stweil
Copy link
Collaborator Author

stweil commented Oct 15, 2020

No, creating a venv does not update pip automatically.

The current Makefile creates the venv in line 141. Line 142 updates pip. I don't understand your no.

@bertsky
Copy link
Collaborator

bertsky commented Oct 15, 2020

We never needed to update from the old 9-something default until recently.

No, creating a venv does not update pip automatically.

The current Makefile creates the venv in line 141. Line 142 updates pip. I don't understand your no.

You are taking this out of context. The question remains: what do we need this change for in the first place?

@stweil
Copy link
Collaborator Author

stweil commented Oct 15, 2020

The question remains: what do we need this change for in the first place?

See the commit message: pip was not upgraded for an existing venv. So for example this sequence would run with an old version of pip and possibly fail:

python3.7 -m venv myvenv
source myvenv/bin/activate
make all

Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
@stweil stweil merged commit e9d9cb2 into OCR-D:master Oct 15, 2020
@stweil stweil deleted the update-pip branch October 15, 2020 11:14
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.

3 participants