nullpo-head / WSL-Hello-sudo

Let's sudo by face recognition of Windows Hello on Windows Subsystem for Linux (WSL). It runs on both WSL 1 and WSL 2. This is a PAM module for Linux on WSL.
MIT License
1.21k stars 46 forks source link

Breaks if WINUSER contains a space #10

Closed jonaskuske closed 3 years ago

jonaskuske commented 3 years ago

install.sh breaks on line 58 if $WINUSER/$PAM_WSL_HELLO_WINPATH contains a space:

./install.sh: line 58: [: /c/Users/Jonas: binary operator expected

Easy to workaround once you notice the issue because you can pass your own install location as part of the setup, but still would be nice if it'd work out of the box :)