Bug#849800: [debhelper-devel] Bug#849800: debhelper: dh_systemd_start --no-start has no effect

2016-12-31 Thread Peter Colberg
Hi Niels,

Will you upload debhelper 10.2.3 soon? This would give sufficient time
for dependent packages to migrate to testing before the full freeze.

Thanks,
Peter



Bug#849800: [debhelper-devel] Bug#849800: debhelper: dh_systemd_start --no-start has no effect

2016-12-31 Thread Peter Colberg
Control: forcemerge 805878 849800

Hi Niels,

On Sat, Dec 31, 2016 at 09:34:00AM +, Niels Thykier wrote:
> From a quick glance, this looks like a duplicate of #805878. Agreed?

Thanks, that’s the bug.

Peter



Bug#849800: [debhelper-devel] Bug#849800: debhelper: dh_systemd_start --no-start has no effect

2016-12-31 Thread Niels Thykier
Peter Colberg:
> Package: debhelper
> Version: 10.2.2
> Severity: normal
> 
> Dear Maintainer,
> 
> I would like to add a systemd service and timer to acmetool that is
> to be explicitly enabled by the user. debian/rules looks as follows:
> 
> ~~~
> [...]
> ~~~
> 
> However, first-time installation of the package yields this warning:
> 
> ~~~
> [...[
> ~~~
> 
> Inspecting the generated postinst it appears --no-start is ignored:
> 
> ~~~
> [...[
> ~~~
> 
> Should the above snippet not be omitted with --no-start?
> 
> Regards,
> Peter
> [...]
> 

Hi Peter,

>From a quick glance, this looks like a duplicate of #805878. Agreed?

Thanks,
~Niels



Bug#849800: debhelper: dh_systemd_start --no-start has no effect

2016-12-30 Thread Peter Colberg
Package: debhelper
Version: 10.2.2
Severity: normal

Dear Maintainer,

I would like to add a systemd service and timer to acmetool that is
to be explicitly enabled by the user. debian/rules looks as follows:

~~~
override_dh_systemd_enable:
dh_systemd_enable --no-enable acmetool.timer

override_dh_systemd_start:
dh_systemd_start --no-start acmetool.timer

override_dh_installinit:
~~~

However, first-time installation of the package yields this warning:

~~~
# dpkg -i acmetool_0.0.58-5_amd64.deb
Selecting previously unselected package acmetool.
(Reading database ... 52232 files and directories currently installed.)
Preparing to unpack .../acmetool_0.0.58-5_amd64.deb ...
Unpacking acmetool (0.0.58-5) ...
Setting up acmetool (0.0.58-5) ...
acmetool.timer is a disabled or a static unit, not starting it.
~~~

Inspecting the generated postinst it appears --no-start is ignored:

~~~
# Automatically added by dh_systemd_start
if [ -d /run/systemd/system ]; then
systemctl --system daemon-reload >/dev/null || true
if [ -n "$2" ]; then
_dh_action=try-restart
else
_dh_action=start
fi
deb-systemd-invoke $_dh_action acmetool.timer >/dev/null || true
fi
# End automatically added section
~~~

Should the above snippet not be omitted with --no-start?

Regards,
Peter