Bug#712167: dbus: add upstart init support

2014-04-01 Thread Cameron Norman
El Tue, 1 de Apr 2014 a las 4:05 AM, Dimitri John Ledkov escribió: On 26 March 2014 10:46, Simon McVittie wrote: On Tue, 25 Mar 2014 at 22:41:32 -0007, Cameron Norman wrote: ‘2’ i can subscribe to dbus package in debian. I feel very stupid because I literally can not figure out how to su

Bug#712167: dbus: add upstart init support

2014-04-01 Thread Dimitri John Ledkov
On 26 March 2014 10:46, Simon McVittie wrote: > On Tue, 25 Mar 2014 at 22:41:32 -0007, Cameron Norman wrote: >> I have attached a deb diff that includes an upstart job. It is different >> from the previous diff by Dimitri because it prompts for a reboot in the >> post installation regardless, per

Bug#712167: dbus: add upstart init support

2014-03-26 Thread Cameron Norman
On Wed, Mar 26, 2014 at 3:46 AM, Simon McVittie wrote: On Tue, 25 Mar 2014 at 22:41:32 -0007, Cameron Norman wrote: > It also uses the --nopidfile option to start dbus. Does the combination of "expect fork" and "there is no pid file" work properly? I'm somewhat surprised if it does; but if Upst

Bug#712167: dbus: add upstart init support

2014-03-26 Thread Simon McVittie
On Tue, 25 Mar 2014 at 22:41:32 -0007, Cameron Norman wrote: > I have attached a deb diff that includes an upstart job. It is different > from the previous diff by Dimitri because it prompts for a reboot in the > post installation regardless, per Simon McVittie's suggestion. That seems sensible. I

Bug#712167: dbus: add upstart init support

2014-03-25 Thread Cameron Norman
I have attached a deb diff that includes an upstart job. It is different from the previous diff by Dimitri because it prompts for a reboot in the post installation regardless, per Simon McVittie's suggestion. It also uses the --nopidfile option to start dbus. Please consider this for inclusion

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Michael Biebl
Am 14.06.2013 16:38, schrieb Simon McVittie: > On 14/06/13 15:09, Michael Biebl wrote:> +if init_is_upstart; then >> +case "$1" in >> +stop) >> +exit 0 >> +;; >> +*) >> +exit 1 >> +;; >> +esac >> +fi >> + >> >> >> I do not like this behaviour at all.

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Simon McVittie
On 14/06/13 15:09, Michael Biebl wrote:> +if init_is_upstart; then > +case "$1" in > + stop) > + exit 0 > + ;; > + *) > + exit 1 > + ;; > +esac > +fi > + > > > I do not like this behaviour at all. > Not only will this break 3rd party tools (monitoring

Bug#712167: [Pkg-utopia-maintainers] Bug#712167: dbus: add upstart init support

2013-06-14 Thread Michael Biebl
Am 13.06.2013 20:23, schrieb Dmitrijs Ledkovs: > Package: dbus > Version: 1.6.12-1 > Severity: normal > Tags: patch > User: upst...@packages.debian.org > Usertags: upstart-job > > Dear Maintainer, > > please consider applying the following patch that adds upstart > integration for the dbus packag

Bug#712167: [Pkg-utopia-maintainers] Bug#712167: dbus: add upstart init support

2013-06-14 Thread Michael Biebl
Am 14.06.2013 14:16, schrieb Simon McVittie: > On 14/06/13 12:54, Dmitrijs Ledkovs wrote: >> On 14 June 2013 12:49, Simon McVittie wrote: >>> Similarly, if you disable or alter the pidfile in the XML, sysvinit >>> and Upstart are not going to work correctly. >> >> Upstart doesn't track pidfiles at

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Simon McVittie
On 14/06/13 12:54, Dmitrijs Ledkovs wrote: > On 14 June 2013 12:49, Simon McVittie wrote: >> Similarly, if you disable or alter the pidfile in the XML, sysvinit >> and Upstart are not going to work correctly. > > Upstart doesn't track pidfiles at all. At the moment it's set to fork, > but could a

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Simon McVittie
On 14/06/13 12:44, Dmitrijs Ledkovs wrote:> On 14 June 2013 12:16, Simon McVittie wrote: >> This is an increasing amount of code to detect whether the dbus-daemon >> is running, so that we know whether to prompt the user to reboot. > > Taking in your suggestion about pgrep/unconditional trigger, h

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Dmitrijs Ledkovs
On 14 June 2013 12:49, Simon McVittie wrote: > On Fri, 14 Jun 2013 at 12:16:28 +0100, Simon McVittie wrote: >> On Thu, 13 Jun 2013 at 19:23:46 +0100, Dmitrijs Ledkovs wrote: >> > The upstart job attached will continue to honour existing >> > settings for additional parameters from /etc/default/dbu

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Simon McVittie
On Fri, 14 Jun 2013 at 12:16:28 +0100, Simon McVittie wrote: > On Thu, 13 Jun 2013 at 19:23:46 +0100, Dmitrijs Ledkovs wrote: > > The upstart job attached will continue to honour existing > > settings for additional parameters from /etc/default/dbus, just like > > the current init.d script. > > Do

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Dmitrijs Ledkovs
On 14 June 2013 12:16, Simon McVittie wrote: > On Thu, 13 Jun 2013 at 19:23:46 +0100, Dmitrijs Ledkovs wrote: >> please consider applying the following patch that adds upstart >> integration for the dbus package. > > pkg-utopia are probably aware, but: dbus/testing has a CVE, so please don't > upl

Bug#712167: dbus: add upstart init support

2013-06-14 Thread Simon McVittie
On Thu, 13 Jun 2013 at 19:23:46 +0100, Dmitrijs Ledkovs wrote: > please consider applying the following patch that adds upstart > integration for the dbus package. pkg-utopia are probably aware, but: dbus/testing has a CVE, so please don't upload dbus/unstable until 1.6.12 has migrated. > The ups

Bug#712167: dbus: add upstart init support

2013-06-13 Thread Dmitrijs Ledkovs
Package: dbus Version: 1.6.12-1 Severity: normal Tags: patch User: upst...@packages.debian.org Usertags: upstart-job -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Dear Maintainer, please consider applying the following patch that adds upstart integration for the dbus package. The patch below