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

Fix #936 #937

Merged
merged 1 commit into from
Apr 13, 2021
Merged

Fix #936 #937

merged 1 commit into from
Apr 13, 2021

Conversation

VEZY
Copy link
Contributor

@VEZY VEZY commented Feb 20, 2021

Adding one more set of single quotes around the url seems to fix the issue. It feels a little hacky, but it's the only way I found to get around this. I tried using Cmd() with windows_verbatim = true too, but it didn't work either.

Adding one more set of single quotes around the url seems to fix the issue. It feels a little hacky, but it's the only way I found to get around this. I tried using `Cmd()` with `windows_verbatim = true` too, but it didn't work either.
@VEZY
Copy link
Contributor Author

VEZY commented Feb 20, 2021

Fix the issue #936, that is to say an issue when using Pluto.run with the notebook argument on windows.
Related to #862, which is closed and resolved but didn't work for me.

@fonsp
Copy link
Owner

fonsp commented Feb 20, 2021

Can you also try to open a notebook with a space in the filename? What about a full path, e.g. "C:\Users\fonsi\notebook.jl"?

@VEZY
Copy link
Contributor Author

VEZY commented Feb 20, 2021

Can you also try to open a notebook with a space in the filename? What about a full path, e.g. "C:\Users\fonsi\notebook.jl"?

Yep good idea! I just tested a file with a space in the filename, a full path with \\, a full path with /, a relative path from a folder above the file, and a path with a folder with a space in it...

...and everything works ! 🎉

@fonsp
Copy link
Owner

fonsp commented Feb 20, 2021

Great, thanks!

Next we need to test on WSL1, WSL2 and maybe another Windows computer, i can do wsl2 and windows

@VEZY
Copy link
Contributor Author

VEZY commented Feb 22, 2021

Great, thanks!

Next we need to test on WSL1, WSL2 and maybe another Windows computer, i can do wsl2 and windows

OK nice ! I just tested WSL1 with Ubuntu 20.04 and it works perfectly.

Base automatically changed from master to main March 11, 2021 21:33
@VEZY
Copy link
Contributor Author

VEZY commented Mar 12, 2021

Hello @fonsp, I had a little bit of time, so I tested on WSL 2 with Ubuntu 20.04 LTS, and it works. I guess that Github actions tests on windows-latest makes for the "another Windows computer" one ?

@stillyslalom
Copy link

Bumping this - it came up in r/Julia. Are there any blockers, or can this be merged?

@fonsp
Copy link
Owner

fonsp commented Apr 7, 2021

@pankgeorg or @ctrekker can you test this on windows?

@fonsp fonsp added help welcome If you are experienced in this topic - let us know! good first issue Good for newcomers labels Apr 7, 2021
@JakobAsslaender
Copy link

I tested it on a windows machine and it works like a charm! Thanks VEZY!

@fonsp fonsp merged commit 8f83657 into fonsp:main Apr 13, 2021
@fonsp
Copy link
Owner

fonsp commented Apr 13, 2021

Thanks for your patience!

@VEZY
Copy link
Contributor Author

VEZY commented Apr 13, 2021

My pleasure, I'm sure you all have a lot of work and that was not a full priority because it worked anyway.

pankgeorg pushed a commit that referenced this pull request Apr 25, 2021
Adding one more set of single quotes around the url seems to fix the issue. It feels a little hacky, but it's the only way I found to get around this. I tried using `Cmd()` with `windows_verbatim = true` too, but it didn't work either.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help welcome If you are experienced in this topic - let us know!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants