Skip to content

optionally create a systemd service file #114

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 1 commit into from
Jan 2, 2020
Merged

optionally create a systemd service file #114

merged 1 commit into from
Jan 2, 2020

Conversation

DerDakon
Copy link
Member

@DerDakon DerDakon commented Jan 1, 2020

This is not installed so packagers can decide in which place they prefer to have it installed.

This should go in before 1.08, so either #76 could also be included, or at least simplify it's packaging after the release.

@DerDakon DerDakon added this to the 1.08 milestone Jan 1, 2020
@mbhangui
Copy link
Contributor

mbhangui commented Jan 1, 2020

For raspberry pi kind of systems which do not have a built in RTC, a problem occurs where the timestamp entries in the log files are totally wrong. I used to have the qmail-send started after the board syncs the time from a NTP source. I don't remember now what I had exactly done and also not sure if anyone would be using qmail on such a system. Just mentioning it because the abrupt change in the times affect the qmailanalog reports. Maybe in the future this could be required to have qmail started only after the time has been set right.
https://www.freedesktop.org/software/systemd/man/systemd-time-wait-sync.service.html

@DerDakon
Copy link
Member Author

DerDakon commented Jan 1, 2020

This generally makes sense, but I personally would leave that as a responsibility of the local admin: if the system needs special workarounds then the admin must know them and how to fix them.

I thought for a moment that it could be useful to also depend on remote-fs.target in case someone has the home directory on a network share. But again, the admin needs to care.

Copy link
Member

@schmonz schmonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise looks fine.

This is not installed so packagers can decide in which place they prefer to have
it installed.
@DerDakon DerDakon merged commit e09fa0b into master Jan 2, 2020
@DerDakon DerDakon deleted the Dakon-systemd branch January 2, 2020 14:35
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.

3 participants