Skip to content

updated the command sequence and location of the service file #1703

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
Jun 8, 2021

Conversation

ronsmits
Copy link
Contributor

The systemctl structure has been changed since 2018. I updated it to the current way of working.

@lurch
Copy link
Contributor

lurch commented Oct 22, 2020

ping @XECDesign

@XECDesign
Copy link
Contributor

Is there some upstream documentation explaining why this path would be preferred? And if so, since which version of systemd?

As far as I can tell /var/run/ is a symlink to /run, which is a tmpfs mount. So anything you put there will disappear after reboot.

That seems to match Redhat's documentation (emphasis mine):

/run/systemd/system/ | Systemd unit files created at run time. This directory takes precedence over the directory with installed service unit files.

The same information is mirrored in upstream documentation:/man/systemd.unit.html

/etc/systemd/system | System units created by the administrator
/run/systemd/system | Runtime units

https://www.freedesktop.org/software/systemd/man/systemd.unit.html

@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

I'd encourage you to wrap this PR up in the next week or so as we're in the process of transitioning the documentation from the current Markdown-based source format to Asciidoc. At some point soon — probably around the end of June, beginning of July — we will freeze the current documentation repo. After that time contributions and PRs based on the Markdown source will not be accepted, and any PRs that are still open will be closed.

See #1911 for full details.

@aallan
Copy link
Contributor

aallan commented Jun 8, 2021

Anyone against merging?

@aallan aallan added the ready to merge The OP says this PR is ready to merge? Anyone object? label Jun 8, 2021
@JamesH65 JamesH65 merged commit 31de976 into raspberrypi:master Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge The OP says this PR is ready to merge? Anyone object?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants