Bug#768609: Please document how to handle .path units

[Luca Boccassi]

Control: tags -1 patch

On Sat, 08 Nov 2014 09:30:54 -0800 Josh Triplett <josh at joshtriplett.org> wrote:
> Package: dh-systemd
> Version: 1.21
> Severity: normal
> File: /usr/bin/dh_systemd_enable
> 
> dh_systemd_enable appears to support the unit types .service and
> ..socket, as well as installing .tmpfile files as tmpfiles.d
> configuration files.  However, the manpage for dh_systemd_enable does
> not document the .socket support, and I don't see any support in
> dh_systemd_enable for any other kind of unit, such as .path.
> 
> In any case, please document in the dh_systemd_enable manpage how to
> handle .path units.
> 
> - Josh Triplett

Dear Maintainer,

Attached is a small patch to add support for installing and
starting/stopping path units. Tested on Jessie.

Thanks!

Kind regards,
Luca Boccassi


From 13e3d38d27cfa47aa000b891528d03f045a34fa1 Mon Sep 17 00:00:00 2001
From: Luca Boccassi <luca.boccassi at gmail.com>
Date: Thu, 26 May 2016 19:26:48 +0100
Subject: [PATCH] Add support for installing path units

---
 script/dh_systemd_enable | 12 +++++++++++-
 script/dh_systemd_start  |  2 +-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/script/dh_systemd_enable b/script/dh_systemd_enable
index cc8f46e..6cd7e3d 100755
--- a/script/dh_systemd_enable
+++ b/script/dh_systemd_enable
@@ -101,7 +101,7 @@ sub contains_install_section {
 	return 0;
 }
 
-# PROMISE: DH NOOP WITHOUT tmp(lib/systemd/system) mount service socket target tmpfile
+# PROMISE: DH NOOP WITHOUT tmp(lib/systemd/system) mount path service socket target tmpfile
 
 foreach my $package (@{$dh{DOPACKAGES}}) {
 	my $tmpdir = tmpdir($package);
@@ -187,6 +187,16 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
 		doit("install","-p","-m644",$mount,"$path/$script.mount");
 	}
 
+	my $pathunit=pkgfile($package,"path");
+	if ($pathunit ne '') {
+		my $path="$tmpdir/lib/systemd/system";
+		if (! -d "$path") {
+			doit("install","-d","$path");
+		}
+
+		doit("install","-p","-m644",$pathunit,"$path/$script.path");
+	}
+
 	find({
 		wanted => sub {
 			my $name = $File::Find::name;
diff --git a/script/dh_systemd_start b/script/dh_systemd_start
index 2d89ea4..2aa5ea9 100755
--- a/script/dh_systemd_start
+++ b/script/dh_systemd_start
@@ -204,7 +204,7 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
 		push @{$aliases{$name}}, $_ for extract_key($name, 'Alias');
 		my @sysv = grep {
 				my $base = $_;
-				$base =~ s/\.(?:mount|service|socket|target)$//g;
+				$base =~ s/\.(?:mount|service|socket|target|path)$//g;
 				-f "$tmpdir/etc/init.d/$base"
 			} ($base, @{$aliases{$name}});
 		if (@sysv == 0 && !grep { $_ eq $name } @units) {
-- 
2.1.4

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/pkg-systemd-maintainers/attachments/20160526/0253268b/attachment.sig>