mirror of
https://github.com/debauchee/barrier.git
synced 2026-05-15 14:16:02 -06:00
[GH-ISSUE #733] systemd barrierc won't start due to (code=exited, status=203/EXEC) in ExecStartPre=/usr/bin/mkdir -p ${FP_DIR} #576
Labels
No labels
HiDPI
bounty
bsd/freebsd
bsd/openbsd
bug
bug
build-infra
cantfix
critical
doc
duplicate
enhancement
fix-available
from git
from release
good first issue
help wanted
installer/package
invalid
linux
macOS
meta
needs testing
pull-request
query
question
regression
regression
v2.4.0
windows
wontfix
work-in-progress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/barrier#576
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jmartens on GitHub (Jun 3, 2020).
Original GitHub issue: https://github.com/debauchee/barrier/issues/733
Operating Systems
Operating Systems
Server: Windows 10 Professional 1909
Client: Ubuntu 20.04 LTS
Barrier Version
Server: 2.3.2-snapshot-210cb270
Client: 2.3.2-snapshot-0a1f0802 (latest master)
Steps to reproduce bug
make installsudo systemctl start barrierc@ip-of-master.service@jmartens commented on GitHub (Jun 3, 2020):
When ExecStartPre steps are done manually it is possible to start the barrier running in client mode using the same command from a terminal window though. Systemd does not allow it apparently.
@jmartens commented on GitHub (Jun 6, 2020):
I am not sure if the complete path to executables are realy required as
ExecStartPre=+mkdir -p "${FP_DIR}andExecStartPre=+sh -c "[ -f "${FP_DIR}/TrustedServers.txt" ] ||\ ...seems to work for me on Ubuntu 20.04 LTS apart from that other executables like openssl also are not referenced using the full path:At least on my system there is not /usr/bin/mkdir, mine is in /bin/
I added the + sign to the commands as PermitRootLogin is deprecated and this is the new method to allow elevation, see systemd docs for ExecStart
@shymega commented on GitHub (Jun 6, 2020):
Agreed. We need to ensure portability.. whilst absolute paths to executables are - to me, preferable on my own systems - for a varying range of distros that Barrier supports, we should avoid absolute paths.
@simons-public commented on GitHub (Jun 6, 2020):
@jmartens @shymega I replaced the
mkdircommands with StateDirectory as recommended in the PR thread. With that change the only relative path should be/bin/sh. Unfortunately older versions of systemd don't support relative paths so it's kind of a necessity here, but/bin/shshould exist on any reasonable system.Adding
+to the/bin/shcommand shouldn't be necessary anymore with StateDirectory since it sets permissions.@github-actions[bot] commented on GitHub (Sep 20, 2020):
This issue has been automatically marked as stale due to inactivity. It will be closed if no further activity occurs. Thank you for your contributions.
@p12tic commented on GitHub (Jan 10, 2021):
@simons-public Should this bug still be open? I'm trying to figure out whether the bot closed this bug unnecessarily.
@p12tic commented on GitHub (Jan 10, 2021):
Let's reopen so that it's not forgotten again. We can close anytime.