Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Systemd service changes #17

Merged
merged 1 commit into from
Mar 6, 2019

Conversation

nicoulaj
Copy link
Contributor

@nicoulaj nicoulaj commented Mar 3, 2019

Here a set of proposed changes regarding the systemd service:

  1. Change to a system instead of a user service. In the general case (single user machine) we want this service to run from boot to shutdown, not only on user session. Using a user service forces users to enable lingering to achieve that. This change also removes the need for specific udev rules / plugdev group.
  2. Enable more hardening options.
  3. Add support for optional /etc/default/linux-thermaltake-rgb file (can contain DEBUG=true).
  4. Add delay between restarts.
  5. Change back executable path to the default /usr/bin. It is downstream packagers job to adapt the service file if they don't deploy to the standard location.

Some of these might be controversial, so open for discussion ;)

 - Change to a system instead of a user service. In the general case (single user machine) we want this service to run from boot to shutdown, not only on user session. Using a user service forces users to enable lingering to achieve that. This change also removes the need for specific udev rules / plugdev group.
 - Enable more hardening options.
 - Add support for optional /etc/default/linux-thermaltake-rgb file (can contain DEBUG=true).
 - Add delay between restarts.
 - Change back executable path to the default /usr/bin. It is downstream packagers job to adapt the service file if they don't deploy to the standard location.

Signed-off-by: Julien Nicoulaud <julien.nicoulaud@gmail.com>
@nicoulaj nicoulaj force-pushed the systemd-system-service branch from 8494ac5 to 6220cb4 Compare March 3, 2019 20:14
@chestm007 chestm007 merged commit eb0d45a into chestm007:master Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants