gsutil error

[terriko]

Hi @terriko on the venv part, I saw the same error for gsutil mentioned here. I.e.:

$ python -m venv venv
$ venv/bin/pip install cve-bin-tool
$ venv/bin/cve-bin-tool ...
...
FileNotFoundError: [Errno 2] No such file or directory: 'gsutil'

On debugging, I noticed that the tool currently expects gsutil on the PATH while I was running my command with an > absolute path for venv and not "activating" it. So, doing the following worked:

$ PATH=venv/bin venv/bin/cve-bin-tool
OR
$ source activate venv/bin/activate
$ cve-bin-tool ...

From a technical perspective, something that could be done here is to automatically add the sys.prefix to the PATH
Or attempt to find gsutil with shutil.which('gsutil') and if not available ... check if {sys.prefix}/bin/gsutil exists ?
Somewhere around: https://github.com/intel/cve-bin-tool/blob/v3.3/cve_bin_tool/data_sources/osv_source.py#L57

Originally posted by @AbdealiLoKo in #2869 (comment)

Moving this out of the original (closed) issue so it might actually get found and fixed by someone.

[terriko]

Now that I've moved this out of the closed thread, some more personal commentary:

• I think not activating a venv is an incredibly bad idea and you should never do this. A large portion of the point of a venv is to do this kind of automatic adjustment of sys.prefix; if you don't activate it that's exactly the part you're missing and you're going to have problems like this all the time.
• I think what we've got documented with respect to venvs is probably sufficient.
• We should NOT automatically adjust sys.prefix, especially not by just searching for a file named gsutil. Modifying your PATH to arbitrary disk locations is potentially very dangerous and would be very bad security practice.

But given that this has come up at least twice now, I do think we could consider catching the FileNotFound error here and giving a more lightly user-friendly message something like "gsutil not found. Did you need to install requirements or activate a venv where gsutil is installed?"

[AbdealiLoKo]

I just got this error again - and completely forgot the previous analysis I had done :D

Thanks for creating this issue as a separate ticket !
I do think a helpful error would be great. I know many people who do not activate venvs when working on a venv - especially members who frequently switch venvs for different python versions