Logcheck devel - Jul 2006 - Bug#377618: logcheck: Please do not abuse debconf

Package: logcheck
Severity: minor

Hi,

You are using two debconf notes which really look like a debconf abuse
(from debconf-devel(7): "It should be used only for important notes that
the user really should see, since debconf will go to great pains to make
sure the user sees it").

The first note (logcheck/install-note) should probably be moved to a
README.Debian file or somewhere else in the documentation. Or you should
really use debconf to configure these two options (report level and
address) instead of displaying a note.
The second note (logcheck/changes) can probably be removed. It no longer
concerns supported transitions in Debian: the version 1.2.39 is in
sarge, and if I understand correctly, it is for transitions from a
version < 1.2.3. However, this note should have been in a NEWS.Debian
file.

Cheers,

-- 
Thomas Huriaux
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url :
http://lists.alioth.debian.org/pipermail/logcheck-devel/attachments/20060710/192b5f3c/attachment.pgp

On Mon, 10 Jul 2006, Thomas Huriaux wrote:
> Hi,
> 
> You are using two debconf notes which really look like a debconf abuse
> (from debconf-devel(7): "It should be used only for important notes
that
> the user really should see, since debconf will go to great pains to make
> sure the user sees it").
> 
> The first note (logcheck/install-note) should probably be moved to a
> README.Debian file or somewhere else in the documentation. Or you should
> really use debconf to configure these two options (report level and
> address) instead of displaying a note.
debconf notes are usually more effective than README's,
also the note is _only_ displayed at level medium so i fail to see the
abuse of the install-note.
> The second note (logcheck/changes) can probably be removed. It no longer
> concerns supported transitions in Debian: the version 1.2.39 is in
> sarge, and if I understand correctly, it is for transitions from a
> version < 1.2.3. However, this note should have been in a NEWS.Debian
> file.
it only shows up on upgrades and will be removed postetch.
currently we may wish to backport latest etch, so such upgrade
handling must stay.

-- 
maks

On Mon, Jul 10, 2006 at 01:32:46PM +0200, Thomas Huriaux
wrote:
> Package: logcheck
> Severity: minor
> 
> Hi,
> 
> You are using two debconf notes which really look like a debconf abuse
> (from debconf-devel(7): "It should be used only for important notes
that
> the user really should see, since debconf will go to great pains to make
> sure the user sees it").
>
> The first note (logcheck/install-note) should probably be moved to a
> README.Debian file or somewhere else in the documentation. Or you should
> really use debconf to configure these two options (report level and
> address) instead of displaying a note.
> The second note (logcheck/changes) can probably be removed. It no longer
> concerns supported transitions in Debian: the version 1.2.39 is in
> sarge, and if I understand correctly, it is for transitions from a
> version < 1.2.3. However, this note should have been in a NEWS.Debian
> file.
I tend to agree with this somewhat.  The note is uncritical.  Anyway, the first
thing users seem to do after installing a new package is to check out
/usr/share/doc/<pkg>/Readme* -- as they should.

The logcheck/changes note should stick around for backporting reasons, as
maks mentioned.

Any other opinions?

-- 
Todd Troxell
http://rapidpacket.com/~xtat