-
Notifications
You must be signed in to change notification settings - Fork 75
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
prestart path should not be near unitfiles #138
Comments
jsoref
added a commit
to jsoref/pleaserun
that referenced
this issue
Dec 27, 2018
Can you help me understand the negative impact of the described problem?
…On Wed, Dec 26, 2018 at 5:48 PM Josh Soref ***@***.***> wrote:
https://github.com/jordansissel/pleaserun/search?q=prestart.sh&unscoped_q=prestart.sh
Files in systemd directories should be things like:
.service, .slice, .target, .wants, .timer, .socket, etc. -- each of which
is a file type defined, managed, and parsed by systemd.
A simple start would be to replace unit_path with prestart_path so that
it's at least vaguely easy to separate.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#138>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAIC6luP5qLYx6md0ZPoRDXQ6N9Bb3P0ks5u9Cb_gaJpZM4ZiOUg>
.
|
The short of it is that Beyond that, systemd at times adds additional file extensions, so essentially it reserves the directories for its own purpose, if you aren't placing a file type that it promises to support, you potentially interfere with a future behavior of systemd. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/jordansissel/pleaserun/search?q=prestart.sh&unscoped_q=prestart.sh
Files in systemd directories should be things like:
.service
,.slice
,.target
,.wants
,.timer
,.socket
, etc. -- each of which is a file type defined, managed, and parsed by systemd.A simple start would be to replace unit_path with prestart_path so that it's at least vaguely easy to separate.
The text was updated successfully, but these errors were encountered: