Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Backslashes in Windows paths need to be escaped for popen to run correctly #50

Open
rs2 opened this issue Aug 2, 2016 · 5 comments
Open
Assignees

Comments

@rs2
Copy link

rs2 commented Aug 2, 2016

Else jinja's FileSystemLoader does not find the template and you get a 500 error:
https://github.com/jupyter-incubator/dashboards_bundlers/blob/master/dashboards_bundlers/local_deploy/__init__.py#L213

replace('\\', '\\\\') is hacky, but is sufficient for Windows non-UNC paths.

@jhpedemonte
Copy link
Collaborator

Assigning to @nitind to investigate a fix.

@nitind
Copy link
Member

nitind commented Aug 16, 2016

@rs which version of Python were you running, and was it from Conda?

@rs2
Copy link
Author

rs2 commented Aug 17, 2016

3.5, but this behavior on Windows is identical on Python > 2.6, including Anaconda distros.
See https://docs.python.org/3/library/os.html#os.sep.

@nitind
Copy link
Member

nitind commented Aug 17, 2016

@rs2 Are you able to try the change in #54 ? It should just be a matter of locating the right __init__.py file in your enabled extensions, modifying it to be https://github.com/nitind/dashboards_bundlers/blob/9e3897db3eb2a32b3dc6a1d3789f022682848202/dashboards_bundlers/local_deploy/__init__.py (this includes the CryptGen fix from #53 ), and removing the __pycache__ directory in the same folder.

@rs2
Copy link
Author

rs2 commented Aug 17, 2016

Let me try

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants