-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
Fix Unicode $USER #2278
Fix Unicode $USER #2278
Conversation
When `$USER` contains Unicode then cmd.exe will return `%AppData%` in Windows Code Page which won't be recognized by Linux/grep. So make sure it outputs result in UTF-8 instead.
You can verify it like this
As you can see in Windows Code Page |
Hi @davispuh, Thanks for the PR. While I'm inclined to improve Unicode support, I can't reproduce this. I've added a user to my machine with a unicode name ( Then, I try to start wine, but wine can't every seem to install mono (or gecko): Fails to install mono (it downloads, but then fails the checksum check. Similar for wine-gecko/wine-9.19. Does this work for you? What wine version? I've never used a unicode username before, is there something else needed? I tried pre-prepopulating ~/.cache/wine with the .msi installers, but no change. |
Are you saying you can't even get Wine to work with such username? I'm on ArchLinux with wine-9.19 and Mono I have installed with Arch Linux package I recommend just using cmd.exe and not Notepad. It's fully reproducible for me
|
Notepad itself will run without it. I was attempting to run the test suite with this change, to make sure it doesn't break anything else, when I noticed the mono/gecko issue. Trying again today, I'm not getting a checksum error (seems like others were, so maybe that was a temporary winehq issue, https://bugs.winehq.org/show_bug.cgi?id=57331). I do notice that it does install, but it doesn't cache the installer (as it does with my normal user). I think the reason you haven't seen this is the Arch wine-mono package installed. If I put the gecko/mono installers in I opened a Wine bug for the installer not caching: https://bugs.winehq.org/show_bug.cgi?id=57338 I'd like to make sure this is something Wine supports before adding a workaround to Winetricks. |
This reverts commit bfb1282.
This reverts commit bfb1282.
This reverts commit bfb1282.
This reverts commit bfb1282.
My
$USER
isDāvis
which contains Unicode and that causes%AppData%
to beC:\users\Dāvis\AppData\Roaming
Currently
winetricks
doesn't work at all for me and fails with:Reason is that because when
cmd.exe
pipes output it will use Windows Code Page which won't be recognized by Linux/grep (it thinks it's binary output). So this PR fixes it by enforcing that output is in UTF-8 instead which works correctly.