systemd updates #175

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
5 participants

jney commented Mar 2, 2013

No description provided.

Owner

kr commented Apr 3, 2016

I was under the impression that port activation would take care of startup, so the WantedBy and After lines are not what we want. Was I wrong?

@@ -3,4 +3,4 @@ Description=Beanstalkd is a simple, fast work queue
[Service]
User=nobody
-ExecStart=/usr/bin/beanstalkd
+ExecStart=/usr/local/bin/beanstalkd
@pprindeville

pprindeville Mar 23, 2017

Why does the path need to change? And why not used the $(PREFIX) in the Makefile, and add a sed command?

I.e. you could change this line to:

ExecStart=@PREFIX@/beanstalkd

and then have in the Makefile:

sed -i -e 's/@PREFIX@/$(PREFIX)/g' ...

when the file gets installed.

-ExecStart=/usr/bin/beanstalkd
+ExecStart=/usr/local/bin/beanstalkd
+
+[Install]
+ExecStart=/usr/local/bin/beanstalkd
+
+[Install]
+WantedBy=multi-user.target

What is this? Could you clarify your comment, please?

@@ -3,4 +3,4 @@ Description=Beanstalkd is a simple, fast work queue
[Service]
User=nobody
-ExecStart=/usr/bin/beanstalkd
+ExecStart=/usr/local/bin/beanstalkd
@pprindeville

pprindeville Mar 23, 2017

Why does the path need to change? And why not used the $(PREFIX) in the Makefile, and add a sed command?

I.e. you could change this line to:

ExecStart=@PREFIX@/beanstalkd

and then have in the Makefile:

sed -i -e 's/@PREFIX@/$(PREFIX)/g' ...

when the file gets installed.

jney commented Mar 26, 2017

i made this PR while i was on archlinux. which explain the path.
i don't use beanstalkd anymore, so i'm not relevant about systemd support. my opinion is, this PR should be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment