dashboards_bundlers icon indicating copy to clipboard operation
dashboards_bundlers copied to clipboard

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

Open rs2 opened this issue 9 years ago • 5 comments

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.

rs2 avatar Aug 02 '16 17:08 rs2

Assigning to @nitind to investigate a fix.

jhpedemonte avatar Aug 02 '16 20:08 jhpedemonte

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

nitind avatar Aug 16 '16 23:08 nitind

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.

rs2 avatar Aug 17 '16 10:08 rs2

@rs2 Are you able to try the change in https://github.com/jupyter-incubator/dashboards_bundlers/pull/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 https://github.com/jupyter-incubator/dashboards_bundlers/pull/53 ), and removing the __pycache__ directory in the same folder.

nitind avatar Aug 17 '16 15:08 nitind

Let me try

rs2 avatar Aug 17 '16 16:08 rs2