Bug#781209: postinst execution order bug confuses systemd

2015-04-04 Thread Romain Francoise
Hi, Okay, so after doing some testing in a virtual machine I'm confident that simply adding the symlink is enough to make the transition from a sysv-managed starter to the native service work as expected. I uploaded version 5.2.1-6 with this fix: https://anonscm.debian.org/cgit/pkg-swan/strongs

Bug#781209: postinst execution order bug confuses systemd

2015-04-01 Thread Michael Biebl
Am 01.04.2015 um 18:23 schrieb Romain Francoise: > On Thu, Mar 26, 2015 at 09:36:32PM +0100, Michael Biebl wrote: >> So I decided to ship a /lib/systemd/system/network-manager.service >> symlink pointing at NetworkManager.service: > >> http://anonscm.debian.org/cgit/pkg-utopia/network-manager.git/

Bug#781209: postinst execution order bug confuses systemd

2015-04-01 Thread Romain Francoise
On Thu, Mar 26, 2015 at 09:36:32PM +0100, Michael Biebl wrote: > So I decided to ship a /lib/systemd/system/network-manager.service > symlink pointing at NetworkManager.service: > http://anonscm.debian.org/cgit/pkg-utopia/network-manager.git/tree/debian/rules#n64 Why do you have a call to dh_syst

Bug#781209: postinst execution order bug confuses systemd

2015-03-30 Thread Romain Francoise
On Sun, Mar 29, 2015 at 06:45:45PM +0200, Michael Biebl wrote: > Can you be more specific, what you have in mind here? Nevermind, I found a machine with Network Manager installed and got the answer to my question: with the symlink, systemd uses the target of the symlink as the real service and add

Bug#781209: postinst execution order bug confuses systemd

2015-03-29 Thread Michael Biebl
Hi Romain, Am 29.03.2015 um 12:56 schrieb Romain Francoise: > On Thu, Mar 26, 2015 at 09:36:32PM +0100, Michael Biebl wrote: >> You could also ship the alias/symlink in the package, and not create it >> via Alias= >> Actually, that's what I would suggest to do anyway to align the old and >> new na

Bug#781209: postinst execution order bug confuses systemd

2015-03-29 Thread Romain Francoise
On Thu, Mar 26, 2015 at 09:36:32PM +0100, Michael Biebl wrote: > You could also ship the alias/symlink in the package, and not create it > via Alias= > Actually, that's what I would suggest to do anyway to align the old and > new name. Thanks for the suggestion, that would probably be more reliabl

Bug#781209: postinst execution order bug confuses systemd

2015-03-26 Thread Michael Biebl
On Thu, 26 Mar 2015 10:41:55 +0100 Romain Francoise wrote: > Hi, > > Thank you for this detailed report, and sorry for the inconvenience... > > On Thu, Mar 26, 2015 at 04:29:02AM +0200, Faidon Liambotis wrote: > > The package's postinst, however, is buggy: it does not use > > dh_installinit but

Bug#781209: postinst execution order bug confuses systemd

2015-03-26 Thread Romain Francoise
Hi, Thank you for this detailed report, and sorry for the inconvenience... On Thu, Mar 26, 2015 at 04:29:02AM +0200, Faidon Liambotis wrote: > The package's postinst, however, is buggy: it does not use > dh_installinit but calls invoke-rc.d ipsec manually. That would have been > fine, but invoke-

Bug#781209: postinst execution order bug confuses systemd

2015-03-25 Thread Faidon Liambotis
Package: strongswan-starter Version: 5.2.1-5 Severity: grave strongswan-starter currently ships: - /etc/init.d/ipsec - /lib/systemd/system/strongswan.service With the latter containing Alias=ipsec.service and also calling the ipsec binary with --nofork as an (implicit) Type=simple unit. This is