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

POSIX filenames and invalid characters on DrvFs #1804

Closed
billziss-gh opened this issue Mar 23, 2017 · 5 comments
Closed

POSIX filenames and invalid characters on DrvFs #1804

billziss-gh opened this issue Mar 23, 2017 · 5 comments

Comments

@billziss-gh
Copy link

A brief description

POSIX allows all characters in filenames except / and \0. However WSL follows Windows guidelines on DrvFs and cannot create filenames that contain any of the NTFS invalid characters: < > : " / \ | ? * or those with values in the range [0, 31].

POSIX filenames should be fully supported. This should be done in a backwards compatible manner as this has been done before by Cygwin and (I believe?) by Services for UNIX: https://www.cygwin.com/cygwin-ug-net/using-specialnames.html

[NOTE: despite my GitHub badge I am not affiliated with Cygwin other than as a user.]

I am aware of #1514.

Expected results

From a Cygwin prompt:

billziss@windows:~$ touch CON
billziss@windows:~$ ls -l CON
-rw-r--r--+ 1 billziss None 0 Mar 23 10:51 CON
billziss@windows:~$ rm CON
billziss@windows:~$ touch C:
billziss@windows:~$ ls -l C:
-rw-r--r--+ 1 billziss None 0 Mar 23 10:51 C:
billziss@windows:~$ rm C:

Actual results (with terminal output if applicable)

From a WSL prompt (on DrvFs):

billziss@WINDOWS:/mnt/c/Users/billziss$ touch CON
billziss@WINDOWS:/mnt/c/Users/billziss$ ls -l CON
-rwxrwxrwx 1 root root 0 Mar 23 10:48 CON
billziss@WINDOWS:/mnt/c/Users/billziss$ rm CON
billziss@WINDOWS:/mnt/c/Users/billziss$ touch C:
touch: setting times of ‘C:’: No such file or directory

Your Windows build number

14393

@therealkenc
Copy link
Collaborator

Same as #1514

@billziss-gh
Copy link
Author

billziss-gh commented Mar 23, 2017

@therealkenc thanks.

I am aware of #1514. However this issue is specifically about using the same filename mapping as Cygwin (and SFU?).

EDIT: @doehrm hinted at the proposed solution in this message

@billziss-gh
Copy link
Author

Is there an actual resolution to this?

@benhillis
Copy link
Member

@billziss-gh - Yes. Judging by the duplicate item linked this is fixed in 1803.

@billziss-gh
Copy link
Author

Thank you for the clarification.

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

No branches or pull requests

4 participants