Change path to wsl path, if on wsl

[SamarthMayya]

@fonsp, I have followed your comments and added some changes in the open function.

[github-actions]

Try this Pull Request!

Open Julia and type:

julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/SamarthMayya/Pluto.jl", rev="fix_wsl_path")
julia> using Pluto

[SamarthMayya]

resolves #763

[fonsp]

Looks good! Can you:

• put this in a separate function
• use try catch to make sure that it never causes a crash
• share a video recording of the new behaviour

Thanks!!

[SamarthMayya]

Looks good! Can you:

• put this in a separate function
• use try catch to make sure that it never causes a crash
• share a video recording of the new behaviour

Thanks!!

I don't use wsl, so is there a way I can do a video recording on linux, such that the changes are used?

[fonsp]

@pankgeorg can you test this on WSL and work with Samarth on finishing the PR?

[rikhuijzer]

Looks good to me

[SamarthMayya]

@pankgeorg could you explain how these changes solve the problem of Pluto not opening when the file is open somewhere else😅

[pankgeorg]

@pankgeorg could you explain how these changes solve the problem of Pluto not opening when the file is open somewhere else😅

They don't; I replied to myself saying I'm not changing this part of the code! The code didn't work because the function wasn't invoked in the right place and didn't remove the \n in the end. This, plus a small reorganization is what you see in my changes!

[SamarthMayya]

Looks good! Can you:

• put this in a separate function
• use try catch to make sure that it never causes a crash
• share a video recording of the new behaviour

Thanks!!

@pankgeorg could you share a video recording of the functionality 😅? Thanks in advance!

[pankgeorg]

Looks good! Can you:

• put this in a separate function
• use try catch to make sure that it never causes a crash
• share a video recording of the new behaviour

Thanks!!

@pankgeorg could you share a video recording of the functionality 😅? Thanks in advance!

[rikhuijzer]

@pankgeorg let's merge?

[fonsp]

Awesome @SamarthMayya!