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

Problem with umlauts in usernames #162

Closed
shochdoerfer opened this issue Dec 19, 2020 · 3 comments · Fixed by #171
Closed

Problem with umlauts in usernames #162

shochdoerfer opened this issue Dec 19, 2020 · 3 comments · Fixed by #171
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Pending Release This is fixed but not yet released. Target: LTS This is targeted to the LTS release. Target: 3.x This is targeted to the 3.x version.

Comments

@shochdoerfer
Copy link

Background Information:

  • Build: 19042.685 (Windows 10 Version 20H2)
  • WSL: Ubuntu 20.04
  • Version: 3.2.1 (installed via PPA)

Describe the bug
I try to create a shortcut with wslusc and it fails because my username contains an umlaut.

To Reproduce
My username created by Windows is StephanHochdörfer - In WSL2 I can commands like this ls -al /mnt/c/Users/StephanHochdörfer/ without any problems.

This is the wslu command I try to execute: wslusc -g -n MyApp /opt/myapp/bin/myapp.sh

This is the output of wslusc:

[warn] wsl.ico not found in Windows directory. Copying right now...
mkdir: cannot create directory '/mnt/c/Users/StephanHochd\224rfer/wslu': No such file or directory
cp: cannot create regular file '/mnt/c/Users/StephanHochd'$'\224''rfer/wslu': No such file or directory
[info] wsl.ico copied. Located at "/mnt/c/Users/StephanHochd�rfer/wslu".
[warn] wsl-term.ico not found in Windows directory. Copying right now...
mkdir: cannot create directory '/mnt/c/Users/StephanHochd\224rfer/wslu': No such file or directory
cp: cannot create regular file '/mnt/c/Users/StephanHochd'$'\224''rfer/wslu': No such file or directory
[info] wsl-term.ico copied. Located at "/mnt/c/Users/StephanHochd�rfer/wslu".
[warn] wsl-gui.ico not found in Windows directory. Copying right now...
mkdir: cannot create directory '/mnt/c/Users/StephanHochd\224rfer/wslu': No such file or directory
cp: cannot create regular file '/mnt/c/Users/StephanHochd'$'\224''rfer/wslu': No such file or directory
[info] wsl-gui.ico copied. Located at "/mnt/c/Users/StephanHochd�rfer/wslu".
[warn] runHidden.vbs not found in Windows directory. Copying right now...
mkdir: cannot create directory '/mnt/c/Users/StephanHochd\224rfer/wslu': No such file or directory
cp: cannot create regular file '/mnt/c/Users/StephanHochd'$'\224''rfer/wslu': No such file or directory
[info] runHidden.vbs copied. Located at "/mnt/c/Users/StephanHochd�rfer/wslu".
mv: cannot move '/mnt/c/Users/STEPHA~1/AppData/Local/Temp/MyApp.lnk' to '/mnt/c/Users/StephanHochd'$'\224''rfer/OneDrive - Company/Desktop': No such file or directory
[info] Create shortcut MyApp.lnk successful

Expected behavior
The files should get created without a problem.

Additional context
It looks like wslvar needs to handle umlauts. When running wslvar -s USERPROFILE I get C:\Users\StephanHochd�rfer as the output.

@patrick330602 patrick330602 added Bug This is either an issue that describes a bug or a PR that resolves a bug Target: 3.x This is targeted to the 3.x version. labels Dec 20, 2020
@patrick330602
Copy link
Member

Thank you for the bug report. I will look into the issue these days.

@patrick330602 patrick330602 added the Status: In Progress This is being worked on and not ready. label Jan 18, 2021
@patrick330602
Copy link
Member

I have identified where the issue is. The fix is in progress now.

@patrick330602
Copy link
Member

Screenshot 2021-02-02 at 1 03 25 PM

Currently fixed, will be part of 3.2.2 release

@patrick330602 patrick330602 added Status: Pending Release This is fixed but not yet released. and removed Status: In Progress This is being worked on and not ready. labels Feb 2, 2021
patrick330602 added a commit that referenced this issue Feb 2, 2021
This was referenced Feb 2, 2021
@patrick330602 patrick330602 added the Target: LTS This is targeted to the LTS release. label Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Pending Release This is fixed but not yet released. Target: LTS This is targeted to the LTS release. Target: 3.x This is targeted to the 3.x version.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants