ceos/unsloth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

install.sh: redirect Windows exes from /dev/null (fix curl|sh stdin drain)

Browse source 
`curl https://unsloth.ai/install.sh | sh` runs install.sh from a pipe, so the
script *is* the shell's stdin. A Windows process launched via WSL interop
(powershell.exe / cmd.exe) inherits that stdin and drains the remaining
piped script, truncating it -- dash then aborts parsing the tail with
"Syntax error: Unterminated quoted string". This surfaced as a non-fatal
"sh: <line>: Unterminated quoted string" near the end of every non-tty
install (e.g. the WoA install.ps1 -> curl|sh flow). Add `</dev/null` to the
three Windows-exe invocations (create_studio_shortcuts' powershell .lnk
writer + the two browser-open helpers) so they cannot consume the script.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Daniel Han 2026-06-04 00:12:53 -07:00
commit 4d9174b9ea
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
install.sh
View file

@ -732,9 +732,9 @@ _open_browser() {
elif grep -qi microsoft /proc/version 2>/dev/null; then
# WSL: xdg-open is unreliable; use Windows browser via PowerShell or cmd
if command -v powershell.exe >/dev/null 2>&1; then
powershell.exe -NoProfile -Command "Start-Process '$_url'" >/dev/null 2>&1 &
powershell.exe -NoProfile -Command "Start-Process '$_url'" </dev/null >/dev/null 2>&1 &
elif command -v cmd.exe >/dev/null 2>&1; then
cmd.exe /c start "" "$_url" >/dev/null 2>&1 &
cmd.exe /c start "" "$_url" </dev/null >/dev/null 2>&1 &
elif command -v xdg-open >/dev/null 2>&1; then
xdg-open "$_url" >/dev/null 2>&1 &
else
@ -1232,7 +1232,7 @@ WSLPS1_EOF
# Convert WSL path to Windows path for powershell.exe
_css_ps1_win=$(wslpath -w "$_css_ps1_tmp" 2>/dev/null)
if [ -n "$_css_ps1_win" ]; then
powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$_css_ps1_win" >/dev/null 2>&1 && _css_created=1
powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$_css_ps1_win" </dev/null >/dev/null 2>&1 && _css_created=1
fi
rm -f "$_css_ps1_tmp"
fi