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

Windows setup.py: relpath breaks when Python and code on different disks #1213

Closed
Jdaviz-Triage-Bot opened this issue Mar 29, 2022 · 4 comments · Fixed by #1218 or #1234
Closed

Windows setup.py: relpath breaks when Python and code on different disks #1213

Jdaviz-Triage-Bot opened this issue Mar 29, 2022 · 4 comments · Fixed by #1218 or #1234
Labels
bug Something isn't working

Comments

@Jdaviz-Triage-Bot
Copy link

Jdaviz-Triage-Bot commented Mar 29, 2022

Reporter: pllim

Cannot run editable install when Python itself is installed on one disk (say, C:) and the source code checkout of this package is on another disk (say, D:). You will get error like this:

ValueError: path is on mount 'D:', start on mount 'C:'

jdaviz/setup.py

Line 134 in 0712a8c

rel_source = os.path.relpath(os.path.abspath(

Example source path: D:\...\jdaviz\share\jupyter\nbconvert\templates\jdaviz-default
Example destination path: C:\...\Miniconda3\envs\py38\share\jupyter\nbconvert\templates

Also see #925 (comment)

🐱


DISCLAIMER: This issue was autocreated by the Jdaviz Issue Creation Bot on behalf of the reporter. If any information is incorrect, please contact Duy Nguyen

@pllim pllim added the bug Something isn't working label Mar 29, 2022
@pllim
Copy link
Contributor

pllim commented Mar 29, 2022

@maartenbreddels , is relpath really necessary?

@maartenbreddels
Copy link
Collaborator

Not needed, but it makes it more flexible, but an abspath could also work. I'm not sure though that windows can do a symlink pointing to a different drive.

@pllim
Copy link
Contributor

pllim commented Mar 30, 2022

I used to think Windows can't do symlink, period, but maybe that has changed... 🤷

@pllim
Copy link
Contributor

pllim commented Apr 6, 2022

#1218 reverted because of 🍎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants