vss-tools icon indicating copy to clipboard operation
vss-tools copied to clipboard

use apt for pipenv install

Open nwesem opened this issue 1 year ago • 3 comments

https://github.com/COVESA/vss-tools/blob/master/README.md#setup-using-venv

didn't work for me on a clean ubuntu 24.04 install, needed to install pipenv using apt so i would suggest changing this

export PIPENV_VENV_IN_PROJECT=1 # will create a local `.venv` in the project, otherwise uses global location
pip install pipenv
export PATH=/home/${USER}/.local/bin:${PATH}
pipenv install --dev # install the development dependencies as well

to

export PIPENV_VENV_IN_PROJECT=1 # will create a local `.venv` in the project, otherwise uses global location
sudo apt update && sudo apt install pipenv
export PATH=/home/${USER}/.local/bin:${PATH}
pipenv install --dev # install the development dependencies as well

nwesem avatar May 06 '24 12:05 nwesem

I think a change like that makes sense, I think that pip in at least newer versions refuse to install things like pipenv globally, i.e. if you use the default Python/Pip. Feel free to create a PR:

erikbosch avatar May 06 '24 12:05 erikbosch

while we're at it, I realized that we are missing dev dependencies. in particular we are missing golang-go and protobuf-compiler to make the tests succeed, do you agree that we should add them in the code block above the one mentioned here @erikbosch

nwesem avatar May 06 '24 13:05 nwesem

We have some info at https://github.com/COVESA/vss-tools/blob/master/README.md#installing-additional-tools It mentions protobuf-compiler there, but not golang-go. Feel free to move that information to some other place if you like.

erikbosch avatar May 06 '24 13:05 erikbosch