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

WSL's "Rename syscall broken for non-empty directories #163" remains in WLinux #193

Closed
mokjpn opened this issue Nov 16, 2018 · 4 comments
Closed
Labels
Bug This is a problem. Unable To Reproduce We cannot reproduce the problem.

Comments

@mokjpn
Copy link

mokjpn commented Nov 16, 2018

I found "settings sync" vs code extension ( https://marketplace.visualstudio.com/items?itemName=Shan.code-settings-sync ) cannot be installed on wLinux.
VSCode produced message that:

EACCES: permission denied, rename '/home/mokada/.vscode/extensions/.shan.code-settings-sync-3.2.0' -> '/home/mokada/.vscode/extensions/shan.code-settings-sync-3.2.0'

I googled the cause of this error, and found a issue in WSL issue:
Rename syscall broken for non-empty directories #163

and found this is already resolved in 2016:
microsoft/WSL#163 (comment)

But in wLinux, I could confirm the issue:

mokada@Think:~$ mkdir -p test-case/a/b
mokada@Think:~$ touch test-case/a/b/awesomesauce.txt
mokada@Think:~$ mv test-case/a/b test-case/
mv: cannot move 'test-case/a/b' to 'test-case/b': Permission denied
mokada@Think:~$

In "Ubuntu" app that can be installed via Windows Store, there is no error:

mokada@Think:~$ mkdir -p test-case/a/b
mokada@Think:~$ touch test-case/a/b/awesomesauce.txt
mokada@Think:~$ mv test-case/a/b/awesomesauce.txt test-case/
mokada@Think:~$

Is there any version difference between wLinux and Ubuntu?

@patrick330602
Copy link

Thank you for the feedback! however, in local testing, I do not have such issue. Can you post your WLinux version and Windows version?

Also, please use template to submit bug report next time, it will make it easier for us to debug

@patrick330602 patrick330602 added Bug This is a problem. Unable To Reproduce We cannot reproduce the problem. labels Nov 16, 2018
@mokjpn
Copy link
Author

mokjpn commented Nov 16, 2018

Sorry missing template.
WLinux version: 1.1.23.0
Windows 10 Pro Version 1809 build 17763.134

@patrick330602
Copy link

patrick330602 commented Nov 16, 2018

What is the shell you are using? I tested bash and zsh with and without rename but they all works.

@mokjpn
Copy link
Author

mokjpn commented Nov 16, 2018

My shell is bash.
I tried to install wLinux on another machine. Then this problem does not occur on another machine.
So the cause might be my machine environment, Sorry for bothering, I will close this issue.

@mokjpn mokjpn closed this as completed Nov 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a problem. Unable To Reproduce We cannot reproduce the problem.
Projects
None yet
Development

No branches or pull requests

2 participants