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 apoi...@debian.org writes: On Fri, 27 Feb 2015 11:20:30 +0200 Apollon Oikonomopoulos apoi...@debian.org 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

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 apoi...@debian.org 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

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 parav...@debian.org 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

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 parav...@debian.org 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

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 gaud...@debian.org 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

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 gaud...@debian.org 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

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

2015-02-05 Thread Gaudenz Steinlin
Hi Gaudenz Steinlin gaud...@debian.org writes: Stephen Gran sg...@debian.org 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

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

2015-02-05 Thread Gaudenz Steinlin
Stephen Gran sg...@debian.org 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

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. Could

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: , |