Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst

2007-01-17 Thread Jonas Smedegaard
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Michael Biebl wrote:
> I agree with Sjoerd. It's definitely not an RC bug.

Why?

Lack of solution is not an argument for lowering severity.



> If you want to control the starting of hal via policy-rc.d, you have to
> do that indirectly via the dbus init script.

How?

On a system with policy-d responding errorcode 101 always, the hal
daemon is started on package update.



> We are basically bound by the limitations of our current init system.

I agree. Which means this bug stays open, not that severity is low.



> As hal depends on dbus, it has to be restarted/stopped when dbus is
> restarted/stopped. That's why we came up with the solution we have now.
> Having the hal start script installed as init script in /etc/init.d/
> would cause more problems than it would solve.

I agree that the current approach is better for most cases, and I do not
suggest to change to a simpler approach.

But still, the current approach does not follow policy, and it does
cause real harm for some.



 - Jonas

- --
* Jonas Smedegaard - idealist og Internet-arkitekt
* Tlf.: +45 40843136  Website: http://dr.jones.dk/

 - Enden er nær: http://www.shibumi.org/eoti.htm
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFFrfUTn7DbMsAkQLgRAlFVAJ9BjCLdS5DD3mNuW+6WNj73pogIHgCeMilb
Xo/DrcX4TJASvgJ5I0Vq34Q=
=CNDa
-END PGP SIGNATURE-



Processed: Re: Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst

2007-01-17 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

> severity 407132 serious
Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst
Severity set to `serious' from `minor'

> thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst

2007-01-16 Thread Sjoerd Simons
severity 407132 minor
thanks,

On Tue, Jan 16, 2007 at 01:29:12PM +0100, Jonas Smedegaard wrote:
> Package: hal
> Version: 0.5.8.1-6
> Severity: serious
> Justification: Policy 9.3.3.2
> 
> 
> The hal postinst restarts a provided daemon directly, without consulting
> policy-rc.d.
> 
> More detailed, it adds a subscript for dbus to maintain, force-reloads dbus
> and then directly restarts that specific subscript (probably to force-restart
> that sub-daemon but no other dbus-maintained daemons).
> 
> 
> Debian Policy 9.3.3.2 states that "package maintainer scripts must use
> `invoke-rc.d' to invoke the `/etc/init.d/*' initscripts, instead of calling
> them directly.".
> 
> 
> I can only interpret the dbus subscript as indirectly being part of the
> `/etc/init.d/*' initscripts.

Maybe, but is is _not_ managed by sysvinit but by the dbus init scripts, thus
it can't use invoke-rc.d. While i agree it's not as nice it coulde be, i don't
see a real solution untill we get an init system that properly supports events

  Sjoerd
-- 
"In order to make an apple pie from scratch, you must first create the
universe."
-- Carl Sagan, Cosmos


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Processed (with 5 errors): Re: Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst

2007-01-16 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

> severity 407132 minor
Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst
Severity set to `minor' from `serious'

> thanks,
Unknown command or malformed arguments to command.

> On Tue, Jan 16, 2007 at 01:29:12PM +0100, Jonas Smedegaard wrote:
Unknown command or malformed arguments to command.

> > Package: hal
Unknown command or malformed arguments to command.

> > Version: 0.5.8.1-6
Unknown command or malformed arguments to command.

> > Severity: serious
Unknown command or malformed arguments to command.

Too many unknown commands, stopping here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#407132: hal: Circumvents invoke-rc.d (and thus policy-rc.d) in postinst

2007-01-16 Thread Jonas Smedegaard
Package: hal
Version: 0.5.8.1-6
Severity: serious
Justification: Policy 9.3.3.2


The hal postinst restarts a provided daemon directly, without consulting 
policy-rc.d.

More detailed, it adds a subscript for dbus to maintain, force-reloads dbus and 
then directly 
restarts that specific subscript (probably to force-restart that sub-daemon but 
no other 
dbus-maintained daemons).


Debian Policy 9.3.3.2 states that "package maintainer scripts must use 
`invoke-rc.d' to invoke 
the `/etc/init.d/*' initscripts, instead of calling them directly.".


I can only interpret the dbus subscript as indirectly being part of the 
`/etc/init.d/*' 
initscripts.


I believe this to be a severe policy violation: Policy promises a policy-rc.d 
setup to be in 
control of which daemons are invoked on a system. Packaging scripts starting 
system daemons by 
invoking scripts meant to be chainloaded by init.d scripts defeats this.


I do not, however, believe this to be release critical: I would prefer Etch to 
ship even with 
this policy.violating bug, rather than without the hal package (which I also 
suspect would 
cripple/pullout Gnome and other major parts of the distribution).

But instead of playing bug severity games, please consult the release team to 
get this bug 
etch-ignore'd.



Kind regards,

Jonas


-- System Information:
Debian Release: 4.0
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: powerpc (ppc)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18-3-powerpc
Locale: LANG=da_DK.UTF-8, LC_CTYPE=da_DK.UTF-8 (charmap=UTF-8)

Versions of packages hal depends on:
ii  adduser 3.101Add and remove users and groups
ii  dbus1.0.2-1  simple interprocess messaging syst
ii  libc6   2.3.6.ds1-10 GNU C Library: Shared libraries
ii  libdbus-1-3 1.0.2-1  simple interprocess messaging syst
ii  libdbus-glib-1-20.71-3   simple interprocess messaging syst
ii  libexpat1   1.95.8-3.4   XML parsing C library - runtime li
ii  libglib2.0-02.12.6-2 The GLib library of C routines
ii  libhal-storage1 0.5.8.1-6Hardware Abstraction Layer - share
ii  libhal1 0.5.8.1-6Hardware Abstraction Layer - share
ii  libusb-0.1-42:0.1.12-2   userspace USB programming library
ii  libvolume-id0   0.103-2  libvolume_id shared library
ii  lsb-base3.1-22   Linux Standard Base 3.1 init scrip
ii  pciutils1:2.2.4-1Linux PCI Utilities
ii  udev0.103-2  /dev/ and hotplug management daemo
ii  usbutils0.72-7   USB console utilities

Versions of packages hal recommends:
ii  eject 2.1.4-2.1  ejects CDs and operates CD-Changer

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]