[pkg-wpa-devel] Bug#360387: Lowering severity (was: reopening)

[Joerg Friedrich]

Reinhard Tartler schrieb am Donnerstag, 13. April 2006 um 05:34:01 +0000:
> retitle 360387 Please install and enable the old init script by default
> severity 360387 minor
> tags 360387 wontfix

Interesting....

> quit
> 
> On Tue, Apr 11, 2006 at 09:13:09AM +0200, Joerg Friedrich wrote:
> > I hereby reopen this bug, since I do not see that this problem is
> > already solved.
> 
> Please stop playing bts pingpong giving other developers a wrong
> impression on this package.

What's the wrong impression. In my impression upgrading from stable to
the new version will stop my wpasupplicant configureation. 
Other package maintainers seems to put lots of work in creating an
upgrade path if they do not support an old configuration.
The new wpasupplicant does not even notify (by debconf).
It will just stop working.

> 
> I've retitled this bug acordingly to what this bug is actually about.
> This is not in any way severity 'grave', but rather 'minor'. Read the
> following definitions on bug severities [1]
> 
> grave: makes the package in question unusable or mostly so, or causes data
> loss, or introduces a security hole allowing access to the accounts of
> users who use the package. 
> 
> minor: a problem which doesn't affect the package's usefulness, and is
> presumably trivial to fix. 
> 
> I don't see how this issue makes this package unusable [2], nor what
> part of existing debian policy this bug currenlty violates (which would
> warrant severity 'serious').

Serverity grave was set before my reopening.
I consider this bug at least 

important
    a bug which has a major effect on the usability of a package,
    without rendering it completely unusable to everyone.

maybe even serious, but I do not know Debian Policy well enough to
judge.


> Since we ship the init script in
> usr/share/doc/wpasupplicant/examples along with ready to copy and paste
> instructions how to activate it, I consider this issue easy to fix on
> both user and maintainer side.

Why don't support both ways of configuring wpa-supplicant?
Just ship the init-script working in /etc/init.d and disable it by
/etc/defaults/wpasupplicant as it is done in the sarge package?
Advantage: you can restart wpasupplicant in the maintainer-scripts on
package update.


> I'm currently not closing this bug since there is currently discussion
> on pkg-wpa-devel at lists.alioth.debian.org how to solve this issue. I
> outlined there a solution which involves introducing a package, which is
> about implementing the requested mode of operation, which was
> undocumented before anyway. Since it is currently unclear if this will
> go into this source packag or another one, I keep this bug open for now.
> 
> [1] http://www.debian.org/Bugs/Developer#severities
> [2] http://blog.zugschlus.de/archives/372-wpa_supplicant.html 

you should also mention 
http://www.joachim-breitner.de/blog/archives/135-Wanted-Code-Deleter.html


> > and do not delete any old config-files!
> 
> It has been renamed with .dpkg-old suffix. If you see a bug in our
> maintainer scripts, feel free to file another bug, but please only with
> exact upgrade path (please really state the involved versions of
> wpasupplicant to reproduce the issue).

if I read the packages pre- and postinst correctly an unmodified init-scipt
will be deleted,

Since the init-script is usually unmodified it gets deleted.

> > > - or at least ask while installation which mode should be used.
> > > - in the worst case you should leave the /etc/init.d file around and
> > >   change the mode of working only via /etc/default. So the admin would
> > >   also need to have to edit /e/d/wpasupplicant
> 
> I strongly object in leaving an init script around, which is not to be
> used by default anyway. This only confuses new user of the package.

common practice, just 'grep -ir enable /etc/defaults'
I'm sure, you also have some packages installed which use this way.

-- 
Jörg Friedrich

There are only 10 types of people:
Those who understand binary and those who don't.