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

add instructions to install a systemd service #12

Merged
merged 1 commit into from Feb 22, 2019

Conversation

Projects
None yet
2 participants
@anarcat
Copy link
Contributor

anarcat commented Feb 19, 2019

I have removed the --verbose option which I tested here because it
doesn't do anything, see #11.

Otherwise this is pretty much what I'm using in production now.

Closes: #10

add instructions to install a systemd service
I have removed the `--verbose` option which I tested here because it
doesn't do anything, see #11.

Otherwise this is pretty much what I'm using in production now.

Closes: #10

@anarcat anarcat referenced this pull request Feb 19, 2019

Closed

systemd support #10

@gares

This comment has been minimized.

Copy link
Owner

gares commented Feb 21, 2019

Thanks very much for this!

I've just one little question, I'm no systemd expert.
I've the impression that the one shot service depends on networking being no, while the timer one does not declare such a need. Is it a limitation of systemd or it can be actually expressed? (I mean "run this every X if Y holds")

@anarcat

This comment has been minimized.

Copy link
Contributor Author

anarcat commented Feb 21, 2019

I've the impression that the one shot service depends on networking being no, while the timer one does not declare such a need. Is it a limitation of systemd or it can be actually expressed? (I mean "run this every X if Y holds")

As we discussed in #10, the network-online.target doesn't actually trigger in user environment, unfortunately. It's purely cosmetic, and somewhat wishful thinking that it would eventually work. We might want to simply remove it.

But that said, I believe it's sufficient to have it in the .service file.

@gares

This comment has been minimized.

Copy link
Owner

gares commented Feb 22, 2019

Oh right, I did forget. Thanks again

@gares gares merged commit e273f08 into gares:master Feb 22, 2019

@anarcat anarcat deleted the anarcat:systemd branch Mar 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.