diff mbox

[16/38] netatalk: systemd support

Message ID 65f17c0367cfd65343d8529bca5be912b0484ed8.1432332802.git.alex.suykov@gmail.com
State Changes Requested
Headers show

Commit Message

Alex Suykov May 23, 2015, 10:05 a.m. UTC
Signed-off-by: Alex Suykov <alex.suykov@gmail.com>
---
 package/netatalk/netatalk.mk      |  8 ++++++++
 package/netatalk/netatalk.service | 10 ++++++++++
 2 files changed, 18 insertions(+)
 create mode 100644 package/netatalk/netatalk.service

Comments

Maxime Hadjinlian July 13, 2015, 8:34 p.m. UTC | #1
Hi Alex, all

On Sat, May 23, 2015 at 12:05 PM, Alex Suykov <alex.suykov@gmail.com> wrote:
> Signed-off-by: Alex Suykov <alex.suykov@gmail.com>
> ---
>  package/netatalk/netatalk.mk      |  8 ++++++++
>  package/netatalk/netatalk.service | 10 ++++++++++
>  2 files changed, 18 insertions(+)
>  create mode 100644 package/netatalk/netatalk.service
>
> diff --git a/package/netatalk/netatalk.mk b/package/netatalk/netatalk.mk
> index a882b3c..c1fe440 100644
> --- a/package/netatalk/netatalk.mk
> +++ b/package/netatalk/netatalk.mk
> @@ -57,4 +57,12 @@ define NETATALK_INSTALL_INIT_SYSV
>                 $(TARGET_DIR)/etc/init.d/S50netatalk
>  endef
>
> +define NETATALK_INSTALL_INIT_SYSTEMD
> +       $(INSTALL) -D -m 644 package/netatalk/netatalk.service \
> +               $(TARGET_DIR)/usr/lib/systemd/system/netatalk.service
> +       mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
You need to fix the path of this link.
> +       ln -sf /usr/lib/systemd/system/netatalk.service \
> +               $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/netatalk.service
> +endef
> +
>  $(eval $(autotools-package))
> diff --git a/package/netatalk/netatalk.service b/package/netatalk/netatalk.service
> new file mode 100644
> index 0000000..e31bfd1
> --- /dev/null
> +++ b/package/netatalk/netatalk.service
> @@ -0,0 +1,10 @@
> +[Unit]
> +Description=Netatalk
> +After=network.target
> +
> +[Service]
> +ExecStart=/usr/sbin/netatalk -d
I don't think you need '-d' if I look at the source in
'etc/netatalk/netatalk.c' I see that '-d' is for debug, why would we
want that ?

Your patch is marked as "Changes Requested" in the patchwork.
> +Restart=always
> +
> +[Install]
> +WantedBy=multi-user.target
> --
> 2.0.3
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

diff --git a/package/netatalk/netatalk.mk b/package/netatalk/netatalk.mk
index a882b3c..c1fe440 100644
--- a/package/netatalk/netatalk.mk
+++ b/package/netatalk/netatalk.mk
@@ -57,4 +57,12 @@  define NETATALK_INSTALL_INIT_SYSV
 		$(TARGET_DIR)/etc/init.d/S50netatalk
 endef
 
+define NETATALK_INSTALL_INIT_SYSTEMD
+	$(INSTALL) -D -m 644 package/netatalk/netatalk.service \
+		$(TARGET_DIR)/usr/lib/systemd/system/netatalk.service
+	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants
+	ln -sf /usr/lib/systemd/system/netatalk.service \
+		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/netatalk.service
+endef
+
 $(eval $(autotools-package))
diff --git a/package/netatalk/netatalk.service b/package/netatalk/netatalk.service
new file mode 100644
index 0000000..e31bfd1
--- /dev/null
+++ b/package/netatalk/netatalk.service
@@ -0,0 +1,10 @@ 
+[Unit]
+Description=Netatalk
+After=network.target
+
+[Service]
+ExecStart=/usr/sbin/netatalk -d
+Restart=always
+
+[Install]
+WantedBy=multi-user.target