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.

Sign up for GitHub

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

Jump to bottom

bootstrap: Fix "sage-venv-config: /doesnotexist/python3: bad interpreter" #31218

Closed
tobiasdiez opened this issue Jan 10, 2021 · 8 comments
Closed

bootstrap: Fix "sage-venv-config: /doesnotexist/python3: bad interpreter" #31218

tobiasdiez opened this issue Jan 10, 2021 · 8 comments
Labels
c: build p: critical / 2 t: enhancement
Milestone
sage-9.3

Comments

@tobiasdiez
Copy link
Contributor

Running ./bootstrap resulted in several warnings of the form

/mnt/d/Programming/sage/src/bin/sage: /mnt/d/Programming/sage/src/bin/sage-venv-config: /doesnotexist/python3: bad interpreter: No such file or directory

This is fixed by using the currently active python instead.

CC: @mkoeppe

Component: build

Author: Matthias Koeppe

Branch/Commit: d97109e

Reviewer: Tobias Diez

Issue created by migration from https://trac.sagemath.org/ticket/31218
@tobiasdiez tobiasdiez added this to the sage-9.3 milestone Jan 10, 2021
@mkoeppe
Copy link
Contributor

mkoeppe commented Jan 10, 2021

comment:2

This is not the right fix. Executing the non-installed copy of sage-venv-config is supposed to fail.

I'll push a new branch

@mkoeppe mkoeppe changed the title Run sage-venv-config in currently active python bootstrap: Fix "sage-venv-config: /doesnotexist/python3: bad interpreter" Jan 10, 2021
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 10, 2021

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

d97109esrc/bin/sage: Do not clobber SAGE_VENV by running the non-installed sage-venv-config script

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 10, 2021

Changed commit from 5d41fa3 to d97109e

@mkoeppe
Copy link
Contributor

mkoeppe commented Jan 10, 2021

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor

mkoeppe commented Jan 10, 2021

comment:5

Try this branch please

@tobiasdiez
Copy link
Contributor Author

comment:7

Thanks, this works like a charm.

@tobiasdiez
Copy link
Contributor Author

Reviewer: Tobias Diez

@vbraun
Copy link
Member

vbraun commented Jan 24, 2021

Changed branch from public/build/sage-venv-python to d97109e

@vbraun vbraun closed this as completed in c28601e Jan 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: build p: critical / 2 t: enhancement
Projects
None yet
Milestone
sage-9.3
Development

No branches or pull requests
3 participants
@vbraun @tobiasdiez @mkoeppe