Bug#775795: [Pkg-puppet-devel] Bug#775795: Patch to use /usr/sbin/service in Debian service-provider

2015-03-01 Thread Apollon Oikonomopoulos
On 10:05 Sun 01 Mar , Stig Sandbeck Mathisen wrote: > Feel free to commit the patch to the packaging repo. Pushed! signature.asc Description: Digital signature

Bug#775795: [Pkg-puppet-devel] Bug#775795: Patch to use /usr/sbin/service in Debian service-provider

2015-03-01 Thread Stig Sandbeck Mathisen
Apollon Oikonomopoulos writes: > On Fri, 27 Feb 2015 11:20:30 +0200 Apollon Oikonomopoulos > wrote: >> The attached patch on top of 3.7.2-2 (hopefully) addresses all of >> these issues (and drops support for pre-2.88 sysv-rc if you don't >> mind). I have not tested it on a sysvinit Jessie syste

Bug#775795: Patch to use /usr/sbin/service in Debian service-provider

2015-02-28 Thread Apollon Oikonomopoulos
On Fri, 27 Feb 2015 11:20:30 +0200 Apollon Oikonomopoulos wrote: > The attached patch on top of 3.7.2-2 (hopefully) addresses all of > these issues (and drops support for pre-2.88 sysv-rc if you don't > mind). I have not tested it on a sysvinit Jessie system though, so if > anyone could do thi

Bug#775795: Patch to use /usr/sbin/service in Debian service-provider

2015-02-27 Thread Apollon Oikonomopoulos
Hi, On Fri, 06 Feb 2015 15:49:17 +0200 Faidon Liambotis wrote: > This seems like a nice approach for status/start/stop/restart but > unfortunately doesn't address enabled?/enable/disable at all. For > starters, puppet seems to call update-rc.d with "defaults", not > "enable". Even "enable", th

Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-07 Thread Cameron Norman
On Fri, 06 Feb 2015 15:49:17 +0200 Faidon Liambotis wrote: > reopen 775795 > thanks > > On 02/01/15 01:03, Gaudenz Steinlin wrote: > > I created a patch to use /usr/sbin/service as suggested earlier in this > > report to start/stop/status/restart services on Debian. I'm a bit > > reluctant to

Bug#775795: [Pkg-puppet-devel] Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-05 Thread Stig Sandbeck Mathisen
Gaudenz Steinlin writes: > Attached is an updated patch that uses a propoer Ruby constant for > /usr/sbin/service. The first patch was botched by my "Pythonistic" > approach to code this. Patch committed. I've tested the packages with autopkgtest as well as manually. I've uploaded the new packag

Bug#775795: [Pkg-puppet-devel] Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-05 Thread Stig Sandbeck Mathisen
Gaudenz Steinlin writes: > Attached is an updated patch that uses a propoer Ruby constant for > /usr/sbin/service. The first patch was botched by my "Pythonistic" > approach to code this. That looks much better, thanks. -- Stig Sandbeck Mathisen signature.asc Description: PGP signature

Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-05 Thread Gaudenz Steinlin
Hi Gaudenz Steinlin writes: > Stephen Gran writes: > >> Hi, >> >> This one time, at band camp, Stig Sandbeck Mathisen said: >>> >>> Hello, >>> >>> Thanks for the patch. It looks like it has the correct solution, using >>> the Debian abstraction layer over the alternative init systems. >>>

Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-05 Thread Gaudenz Steinlin
Stephen Gran writes: > Hi, > > This one time, at band camp, Stig Sandbeck Mathisen said: >> >> Hello, >> >> Thanks for the patch. It looks like it has the correct solution, using >> the Debian abstraction layer over the alternative init systems. >> >> However, I've found a problem with it usi

Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-05 Thread Stephen Gran
Hi, This one time, at band camp, Stig Sandbeck Mathisen said: > > Hello, > > Thanks for the patch. It looks like it has the correct solution, using > the Debian abstraction layer over the alternative init systems. > > However, I've found a problem with it using the "puppet resource" > command.

Bug#775795: Patch to use /usr/sbin/service in Debian service provider

2015-02-04 Thread Stig Sandbeck Mathisen
Hello, Thanks for the patch. It looks like it has the correct solution, using the Debian abstraction layer over the alternative init systems. However, I've found a problem with it using the "puppet resource" command. Could you see if you find what causes it? With puppet 3.7.22-1: , | root