Add venv for Arch Linux #1382
Open
Add venv for Arch Linux #1382
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bindertron
reviewed
Feb 7, 2025
README.md
Outdated
| @@ -207,6 +215,24 @@ python mtk.py multi "cmd1;cmd2" | |||
| ``` | |||
| See the file "[run.example](https://github.com/bkerler/mtkclient/blob/main/examples/run.example)" on how to structure the script file | |||
|
|
|||
| ### Using in on ARCH (BTW) | |||
There was a problem hiding this comment.
I would go with "Using with virtualenv", because this works on any distro, not just on arch.
README.md
Outdated
|
|
||
| If you gonna run any command, you NEED to use this prefix... | ||
| ``` | ||
| ~/.venv/bin/python mtk.py ... |
There was a problem hiding this comment.
Since this is for non knowledgeable users, I would start with:
source .venv/bin/activate
pip install .And I would not mention the gui when using virtualenv.
Also, once inside virtualenv, you can just run using mtk, see:
mtk --helpTo exit virtualenv:
deactivateThere was a problem hiding this comment.
actually, switching to virtualenv is better than using the same prefix everytime!
|
I'd say people using linux should know how to use venv. A better option is to go with pyenv imho. In any way, @bindertron is right with his comments, please make sure to adapt the PR accordingly. Thanks |
|
fixed it |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Title says it all.