Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-08 Thread Julien BLACHE
Russ Allbery [EMAIL PROTECTED] wrote:

Hi,

 Also, the warning could include the patch name as a useful hint :)

 Yeah, it actually was -- that's how I figured out the problem.  It was
 warning about the patch .  :)

Ah, nice, then :)

JB.

-- 
 Julien BLACHE [EMAIL PROTECTED]  |  Debian, because code matters more 
 Debian  GNU/Linux Developer|   http://www.debian.org
 Public key available on http://www.jblache.org - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Julien BLACHE
Package: lintian
Version: 1.23.38
Severity: normal

Hi,

It looks like lintian gives a bogus warning for dpatch-missing-description
because of the 00list index file:

% ls debian/patches
00list  10_no_lib64_on_64_bits.dpatch  11_always_smp.dpatch  
12_acc_enable_radius_diameter.dpatch

% grep DP: debian/patches/*
debian/patches/10_no_lib64_on_64_bits.dpatch:## All lines beginning with `## 
DP:' are a description of the patch.
debian/patches/10_no_lib64_on_64_bits.dpatch:## DP: Debian uses /usr/lib even 
for 64bit architectures.
debian/patches/11_always_smp.dpatch:## All lines beginning with `## DP:' are a 
description of the patch.
debian/patches/11_always_smp.dpatch:## DP: Always build for SMP.
debian/patches/12_acc_enable_radius_diameter.dpatch:## All lines beginning with 
`## DP:' are a description of the patch.
debian/patches/12_acc_enable_radius_diameter.dpatch:## DP: Enable RADIUS and 
Diameter in the acc module.

Or am I missing something ?

JB.

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.23-hrt1 (SMP w/2 CPU cores)
Locale: LANG=C, [EMAIL PROTECTED] (charmap=ISO-8859-15)
Shell: /bin/sh linked to /bin/bash

Versions of packages lintian depends on:
ii  binutils2.18.1~cvs20071027-2 The GNU assembler, linker and bina
ii  diffstat1.45-2   produces graph of changes introduc
ii  dpkg-dev1.14.12  package building tools for Debian
ii  file4.21-3   Determines file type using magic
ii  gettext 0.17-1   GNU Internationalization utilities
ii  intltool-debian 0.35.0+20060710.1Help i18n of RFC822 compliant conf
ii  libparse-debianchan 1.1.1-1  parse Debian changelogs and output
ii  man-db  2.5.0-4  on-line manual pager
ii  perl [libdigest-md5 5.8.8-12 Larry Wall's Practical Extraction 

lintian recommends no packages.

-- no debconf information



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Russ Allbery
Julien BLACHE [EMAIL PROTECTED] writes:
 Russ Allbery [EMAIL PROTECTED] wrote:

 It's the blank line in 00list.

 Will be fixed in the next upload.

 Great, thanks !

 Also, the warning could include the patch name as a useful hint :)

Yeah, it actually was -- that's how I figured out the problem.  It was
warning about the patch .  :)

-- 
Russ Allbery ([EMAIL PROTECTED])   http://www.eyrie.org/~eagle/



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Julien BLACHE
Russ Allbery [EMAIL PROTECTED] wrote:

Hi,

 Patches for both packages have a description. I haven't tested my
 other packages but I guess the result would be the same.

 It's the blank line in 00list.

 Will be fixed in the next upload.

Great, thanks !

Also, the warning could include the patch name as a useful hint :)

JB.

-- 
 Julien BLACHE [EMAIL PROTECTED]  |  Debian, because code matters more 
 Debian  GNU/Linux Developer|   http://www.debian.org
 Public key available on http://www.jblache.org - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Russ Allbery
Julien BLACHE [EMAIL PROTECTED] writes:

 I encountered that issue first on openser (in experimental), but also
 on iaxmodem (in 1.0.0~dfsg-1, incoming or unstable).

 Patches for both packages have a description. I haven't tested my
 other packages but I guess the result would be the same.

It's the blank line in 00list.

Will be fixed in the next upload.

-- 
Russ Allbery ([EMAIL PROTECTED])   http://www.eyrie.org/~eagle/



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Russ Allbery
Julien BLACHE [EMAIL PROTECTED] writes:

 Package: lintian
 Version: 1.23.38
 Severity: normal

 It looks like lintian gives a bogus warning for
 dpatch-missing-description because of the 00list index file:

Looking at the code, it looks like it should do the right thing, but this
certainly sounds broken.  Is this package in the archive, and if so,
what's its name?  I'll take a look.

-- 
Russ Allbery ([EMAIL PROTECTED])   http://www.eyrie.org/~eagle/



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



Bug#454730: lintian: bogus W: dpatch-missing-description

2007-12-07 Thread Julien BLACHE
Russ Allbery [EMAIL PROTECTED] wrote:

Hi,

 It looks like lintian gives a bogus warning for
 dpatch-missing-description because of the 00list index file:

 Looking at the code, it looks like it should do the right thing, but this
 certainly sounds broken.  Is this package in the archive, and if so,
 what's its name?  I'll take a look.

I encountered that issue first on openser (in experimental), but also
on iaxmodem (in 1.0.0~dfsg-1, incoming or unstable).

Patches for both packages have a description. I haven't tested my
other packages but I guess the result would be the same.

Thanks,

JB.

-- 
 Julien BLACHE [EMAIL PROTECTED]  |  Debian, because code matters more 
 Debian  GNU/Linux Developer|   http://www.debian.org
 Public key available on http://www.jblache.org - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



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