Skip to content

Commit

Permalink
Do not quote $SHELL in .desktop files
Browse files Browse the repository at this point in the history
This does not work properly on some desktop environments (KDE), and
$SHELL is unlikely to require quoting.

Fixes #4367 <#4367>
PR #4448 <#4448>

Signed-off-by: Romain Vimont <[email protected]>
  • Loading branch information
kidonng authored and rom1v committed Nov 25, 2023
1 parent 8db4e78 commit 8976121
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/data/scrcpy-console.desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Comment=Display and control your Android device
# For some users, the PATH or ADB environment variables are set from the shell
# startup file, like .bashrc or .zshrc… Run an interactive shell to get
# environment correctly initialized.
Exec=/bin/sh -c "\"\$SHELL\" -i -c scrcpy --pause-on-exit=if-error"
Exec=/bin/sh -c "\\$SHELL -i -c scrcpy --pause-on-exit=if-error"
Icon=scrcpy
Terminal=true
Type=Application
Expand Down
2 changes: 1 addition & 1 deletion app/data/scrcpy.desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Comment=Display and control your Android device
# For some users, the PATH or ADB environment variables are set from the shell
# startup file, like .bashrc or .zshrc… Run an interactive shell to get
# environment correctly initialized.
Exec=/bin/sh -c "\"\$SHELL\" -i -c scrcpy"
Exec=/bin/sh -c "\\$SHELL -i -c scrcpy"
Icon=scrcpy
Terminal=false
Type=Application
Expand Down

0 comments on commit 8976121

Please sign in to comment.