Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix 4388 to pipenv graph on windows #4409

Merged
merged 3 commits into from Aug 5, 2020
Merged

Conversation

yuzhujiutian
Copy link
Contributor

@yuzhujiutian yuzhujiutian commented Aug 2, 2020

Thank you for contributing to Pipenv!

The issue

F:\helloc\pipenv_proj\warpy_airtestpoco>PIPENV_EENV_IN_PROJECT\warpy_airtestpoco-OqiTcr_4\Scripts\pipenv graph
[PipenvCmdError]: File "f:\helloc\pipenv_proj\warpy_airtestpoco\pipenv_eenv_in_project\warpy_airtestpoco-oqitcr_4\lib\site-packages\pipenv\vendor\click\core.py", line 1259, in invoke
......
[PipenvCmdError]: File "f:\helloc\pipenv_proj\warpy_airtestpoco\pipenv_eenv_in_project\warpy_airtestpoco-oqitcr_4\lib\site-packages\pipenv\utils.py", line 161, in run_command
[PipenvCmdError]: raise PipenvCmdError(cmd_string, c.out, c.err, return_code)
Error running command: $ PIPENV_EENV_IN_PROJECT/warpy_airtestpoco-OqiTcr_4/Scripts/python.exe f:/helloc/pipenv_proj/warpy_airtestpoco/pipenv_eenv_in_project/warpy_airtestpoco-oqitcr_4/lib/site-packages/pipenv/vendor/pipdeptree.py -l
STDERR: 'PIPENV_EENV_IN_PROJECT' 不是内部或外部命令,也不是可运行的程序或批处理文件。

#4388

The fix

By debugging the code, i found that there is no need to call the function as_posix () for python_path and pipdeptree_path on the windows platform.

python_path = vistir.compat.Path(python_path).as_posix()
pipdeptree_path = vistir.compat.Path(pipdeptree_path).as_posix()

The checklist

image

The test

image

@frostming
Copy link
Contributor

Please add a news fragment under the news/ directory.

@frostming frostming added the Status: Awaiting Update ⏳ This issue requires more information before assistance can be provided. label Aug 5, 2020
@frostming frostming merged commit 3dd65c5 into pypa:master Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Awaiting Update ⏳ This issue requires more information before assistance can be provided.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants