Skip to content

Fix systemd comment syntax #10066

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

Merged
merged 2 commits into from
Oct 14, 2020
Merged

Fix systemd comment syntax #10066

merged 2 commits into from
Oct 14, 2020

Conversation

HolgerHees
Copy link
Contributor

comments for systemd service files are only allowed in the beginning of a new line
https://www.freedesktop.org/software/systemd/man/systemd.syntax.html

otherwise you get error messages like

/usr/lib/systemd/system/netdata.service:48: Failed to parse capability in bounding/ambient set, ignoring: #
Summary
Component Name
Test Plan
Additional Information

comments for systemd service files are only allowed in the beginning of a new line
https://www.freedesktop.org/software/systemd/man/systemd.syntax.html

otherwise you get error messages like
```
/usr/lib/systemd/system/netdata.service:48: Failed to parse capability in bounding/ambient set, ignoring: #
```
@HolgerHees HolgerHees requested a review from Ferroin as a code owner October 14, 2020 08:28
@CLAassistant
Copy link

CLAassistant commented Oct 14, 2020

CLA assistant check
All committers have signed the CLA.

@ilyam8
Copy link
Member

ilyam8 commented Oct 14, 2020

@HolgerHees thx for the contribution 👍 There is CLA, sign it!

@github-actions github-actions bot added the area/packaging Packaging and operating systems support label Oct 14, 2020
@Ferroin Ferroin merged commit ba85771 into netdata:master Oct 14, 2020
@Ferroin
Copy link
Member

Ferroin commented Oct 14, 2020

@HolgerHees Thanks for the PR, this will be in the release that goes out today, and should show up in the nightlies tomorrow.

@joelhans This needs to be added to the release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants