Skip to content

Temp directory makes assumption about WSL paths #110

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

Closed
vcsjones opened this issue Jun 4, 2018 · 2 comments · Fixed by #200
Closed

Temp directory makes assumption about WSL paths #110

vcsjones opened this issue Jun 4, 2018 · 2 comments · Fixed by #200

Comments

@vcsjones
Copy link

vcsjones commented Jun 4, 2018

WSL starting in Windows 1803 allows file systems to be mounted in places other than /mnt by configuration.

I've taken to mounting them simply in / so that I can access /c/window-stuff. For example:

[automount]
enabled = true
root = /

The causes the launcher to fail to make a temp directory due to this line:

const userRegExp = /\/mnt\/([a-z])\/Users\/([^\/:]+)\/AppData\//;

It makes an assumption about where the windows file system is mounted.

@vcsjones
Copy link
Author

vcsjones commented Jun 4, 2018

I'm happy to make a pull request to address the issue if anyone has a good idea on how to address this.

@paulirish
Copy link
Member

yah would be happy to get a PR for this.

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

Successfully merging a pull request may close this issue.

2 participants