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

Typo on systemd service units file #2

Merged
merged 1 commit into from
Mar 11, 2017
Merged

Typo on systemd service units file #2

merged 1 commit into from
Mar 11, 2017

Conversation

edmundoferreira
Copy link
Contributor

-Bug
the following commands

systemctl enable simple-daemon.service
systemctl enable forking-daemon.service

generate the following error message
"Failed to execute operation: Invalid argument"

-Actions
corrected typo on WantedBy from multiuser.agent to multi-user.target in both service files

-Fixes
this allows the following command to work properly

  systemctl enable simple-daemon.service
  systemctl enable forking-daemon.service

- the following commands
  systemctl enable simple-daemon.service
  systemctl enable forking-daemon.service
generate the following error message
  "Failed to execute operation: Invalid argument"

Actions
- corrected typo on WantedBy from multiuser.agent to multi-user.target in both service files

Fixes
- this allows the following command to work properly
  systemctl enable simple-daemon.service
  systemctl enable forking-daemon.service
@jirihnidek jirihnidek merged commit 41692c8 into jirihnidek:master Mar 11, 2017
@jirihnidek
Copy link
Owner

Thanks for bug fix. 👍

@edmundoferreira
Copy link
Contributor Author

Your welcome.
Thanks for the great code example on how to daemonize an unix application, one of the best I found online.

I just noticed now that the typo is also in the README.md, sorry forgot to correct that one.

@jirihnidek
Copy link
Owner

No problem :-), just fixed typo in README.md.

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