Allow querying non-host python intepreters

[gaborbernat]

This way you can install once (for example via pipx) and reuse it for any existing python environment.

$ pipdeptree --python venv/bin/python
pip==20.1.1
setuptools==49.2.0
wheel==0.34.2

@naiquevin for your attention. The CI fails because I cannot add the virtualenv dependency to Travis, it will only take effect post merge.

[gaborbernat]

@naiquevin can you take a look at this please?

[naiquevin]

Hi Bernát, Sorry for the delay. Will try to check this PR sometime next week. A bit caught up with work at the moment. Thanks.

On Tue, 4 Aug 2020 at 8:52 PM, Bernát Gábor ***@***.***> wrote: @naiquevin <https://github.com/naiquevin> can you take a look at this please? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#128 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEOZHBQRDAGYR6G3MZLZMDR7ARR3ANCNFSM4PGWVGCA> .

-- ~ Vineet

[naiquevin]

@gaborbernat: That's a clever approach. This fixes a long pending feature request. Thanks a lot!

The code LGTM, but I've added a minor comment.

Also, can you please check the travis failure? It seems to be related to virtualenv version. Possibly related: Issue for travis to support specific virtualenv versions - travis-ci/travis-ci#8342

[naiquevin]

Another minor concern - It's possible that the main pipdeptree process is run using Python 3 but the specified Python interpreter is of version 2. This is not a problem as long as pipdeptree has compatibility with both Python versions (which is the case currently but can't be guaranteed in future). May be there should be a warning/recommendation regarding this in the README.

[gaborbernat]

Another minor concern - It's possible that the main pipdeptree process is run using Python 3 but the specified Python interpreter is of version 2. This is not a problem as long as pipdeptree has compatibility with both Python versions (which is the case currently but can't be guaranteed in future). May be there should be a warning/recommendation regarding this in the README.

IMHO we should add that warning when we drop the python 2 support.

[gaborbernat]

@naiquevin Travis is ignoring our install commands fro some odd reason, so moved to Github Actions within #133 to fix it

[gaborbernat]

Added a commit to fix the --python= format.

[naiquevin]

LGTM. Please rebase.

[gaborbernat]

@naiquevin rebased, and fixed tests that were failing 👍 should be ready for merge.

[gaborbernat]

@naiquevin should be good to go now.

[naiquevin]

Thanks 👍

[gaborbernat]

@naiquevin can we get a new release?

[naiquevin]

Yeah, a release is long pending. I'll try to find some time in the coming week.

[gaborbernat]

@naiquevin Hello any updates, thanks!

[naiquevin]

I was planning to include a PR. But haven't been able to find the time. I guess I'll proceed with creating a new version this weekend without the PR

…

On Mon, Dec 21, 2020 at 9:28 PM Bernát Gábor ***@***.***> wrote: @naiquevin <https://github.com/naiquevin> Hello any updates, thanks! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#128 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEOZHFOPSJKRWF5WI5WVELSV5WCLANCNFSM4PGWVGCA> .

-- ~ Vineet

[gaborbernat]

Thanks that would be much appreciated 👍