Skip to content

Commit

Permalink
Avoid unnecessary fork in the desktop command
Browse files Browse the repository at this point in the history
  • Loading branch information
nurupo committed Jun 18, 2022
1 parent ea6568a commit 039ebc9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ That assumes that the VPN is already up, otherwise `vpn exec` would error and th
If you want to also bring the VPN up at the same time, just in case it's not up already, you can use the following instead:

```
/usr/local/bin/pkexec-E /bin/sh -c '/usr/local/bin/vpn up vpn-name region-id > /dev/null ; /usr/local/bin/vpn exec vpn-name app "$@"' -- %U
/usr/local/bin/pkexec-E /bin/sh -c '/usr/local/bin/vpn up vpn-name region-id > /dev/null ; exec /usr/local/bin/vpn exec vpn-name app "$@"' -- %U
```

`> /dev/null` is needed because there is no tty when the vpn script is run, making echo commands fail to write, killing the script.
Expand Down

0 comments on commit 039ebc9

Please sign in to comment.