Skip to content

Use python variable for creating venv#65

Merged
Crivella merged 1 commit into
Crivella:masterfrom
TROCADER:master
Mar 28, 2026
Merged

Use python variable for creating venv#65
Crivella merged 1 commit into
Crivella:masterfrom
TROCADER:master

Conversation

@TROCADER

Copy link
Copy Markdown
Contributor

When running the script on Linux using sh for the first time it could not find python on line 38, which led to unable to run the command on line 40, as seen below:

PYTHON=python3 sh ./run-user.sh
./run.sh: line 38: python: command not found
Creating venv in directory ./venv using python 
Unable to create venv in directory ./venv

exit code: 0

stderr:
./run.sh: line 40: -m: command not found
`ocr_translate` not found: installing django-ocr_translate...
Installing ocr_translate 0.7.4...

The issue stems from not using the python variable when creating a venv, hence the problem is not present when one is already created.

The minor change to instead use the variable fixed the problem.

@Crivella

Copy link
Copy Markdown
Owner

Thanks for catching this and opening a PR @TROCADER .
Always tested this on systems where python3 was aliased to python or viceversa so it never came up

@Crivella Crivella merged commit 274bad8 into Crivella:master Mar 28, 2026
3 checks passed
@Crivella Crivella added the bug-fix Fixing a bug label Mar 28, 2026
@Crivella Crivella added this to the v0.7.5 milestone Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants