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

config/ndo2db.cfg-sample.in: use @piddir@ for the pid file. #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

orlitzky
Copy link
Contributor

(Related to #71, but useful independently)

The "lock_file" setting in ndo2db.cfg specifies where the daemon's pid file should be stored. In the past, it was stored in @localstatedir@, but @piddir@ is more appropriate. As evidence, all of the init scripts in the "startup" directory reference @piddir@ and not @localstatedir@ for the location of the pid file. This commit updates the sample config to agree with the init scripts.

The "lock_file" setting in ndo2db.cfg specifies where the daemon's pid
file should be stored. In the past, it was stored in @localstatedir@,
but @Piddir@ is more appropriate. As evidence, all of the init scripts
in the "startup" directory reference @Piddir@ and not @localstatedir@
for the location of the pid file. This commit updates the sample
config to agree with the init scripts.
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.

1 participant