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: Patch to use /usr/sbin/service in Debian service provider

2015-02-06 Thread Faidon Liambotis
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 just NMU puppet and would prefer if one of the maintainers and/or Faidon could

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

2015-02-06 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org: reopen 775795 Bug #775795 {Done: Stig Sandbeck Mathisen s...@debian.org} [puppet] puppet: Service's debian provider assumes SysV init 'reopen' may be inappropriate when a bug has been closed with a version; all fixed versions will be cleared, and

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

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

2015-01-31 Thread Gaudenz Steinlin
Control: tags -1 +patch Hi 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 just NMU puppet and would prefer if one of the maintainers and/or Faidon could have a look at my patch first. If you