lintian: r1131 - in trunk: debian frontend testset testset/manpages/debian

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 02:26:03 +0100 (Tue, 22 Jan 2008)
New Revision: 1131

Modified:
   trunk/debian/changelog
   trunk/frontend/lintian
   trunk/testset/manpages/debian/overrides
   trunk/testset/manpages/debian/rules
   trunk/testset/tags.manpages
Log:
* frontend/lintian:
  + [RA] Include info tags in the broken-down description of the
override count since they're included in the total.  Based on a
patch by Chris Lamb.  (Closes: #459851)

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-20 20:54:50 UTC (rev 1130)
+++ trunk/debian/changelog  2008-01-22 01:26:03 UTC (rev 1131)
@@ -11,11 +11,16 @@
   * checks/menu-format:
 + [RA] Avoid a Perl warning for desktop entries without Exec.
 
+  * frontend/lintian:
++ [RA] Include info tags in the broken-down description of the
+  override count since they're included in the total.  Based on a
+  patch by Chris Lamb.  (Closes: #459851)
+
   * lib/Spelling.pm:
 + [RA] Add a spelling correction for PostgreSQL and lots of new
   capitalization corrections after a debian-devel thread.
 
- -- Russ Allbery [EMAIL PROTECTED]  Mon, 07 Jan 2008 23:58:47 -0800
+ -- Russ Allbery [EMAIL PROTECTED]  Mon, 21 Jan 2008 17:25:43 -0800
 
 lintian (1.23.42) unstable; urgency=low
 

Modified: trunk/frontend/lintian
===
--- trunk/frontend/lintian  2008-01-20 20:54:50 UTC (rev 1130)
+++ trunk/frontend/lintian  2008-01-22 01:26:03 UTC (rev 1131)
@@ -1363,15 +1363,17 @@
}
}
 
-# Report override statistics.
-if (not $no_override and not $show_overrides) {
-my $stats = Tags::get_stats($file);
-my $short = $file;
-$short =~ s%.*/%%;
-my $errors = $stats-{overrides}{by_severity}{4} || 0;
-my $warnings = $stats-{overrides}{by_severity}{2} || 0;
+   # Report override statistics.
+   if (not $no_override and not $show_overrides) {
+   my $stats = Tags::get_stats($file);
+   my $short = $file;
+   $short =~ s%.*/%%;
+   my $errors = $stats-{overrides}{by_severity}{4} || 0;
+   my $warnings = $stats-{overrides}{by_severity}{2} || 0;
+   my $info = $stats-{overrides}{by_severity}{0} || 0;
$overrides{errors} += $errors;
$overrides{warnings} += $warnings;
+   $overrides{info} += $info;
 }
 }
 
@@ -1424,24 +1426,23 @@
 if (not $quiet and not $no_override and not $show_overrides) {
 my $errors = $overrides{errors};
 my $warnings = $overrides{warnings};
-my $total = $errors + $warnings;
+my $info = $overrides{info};
+my $total = $errors + $warnings + $info;
 if ($total  0) {
my $total = ($total == 1)
? $total tag overridden
: $total tags overridden;
+   my @output;
if ($errors) {
-   $errors = ($errors == 1) ? $errors error : $errors errors;
+   push (@output, ($errors == 1) ? $errors error : $errors errors);
}
if ($warnings) {
-   $warnings = ($warnings == 1) ? $warnings warning : $warnings 
warnings;
+   push (@output, ($warnings == 1) ? $warnings warning : $warnings 
warnings);
}
-   if ($errors and $warnings) {
-   print N: $total ($errors, $warnings)\n;
-   } elsif ($errors) {
-   print N: $total ($errors)\n;
-   } elsif ($warnings) {
-   print N: $total ($warnings)\n;
+   if ($info) {
+   push (@output, $info info);
}
+   print N: $total (, join (', ', @output), )\n;
 }
 }
 

Modified: trunk/testset/manpages/debian/overrides
===
--- trunk/testset/manpages/debian/overrides 2008-01-20 20:54:50 UTC (rev 
1130)
+++ trunk/testset/manpages/debian/overrides 2008-01-22 01:26:03 UTC (rev 
1131)
@@ -7,3 +7,6 @@
 #and some empty lines
 but-this-is-just-broken!
 manpages:
+
+# An 'info' tag:
+package-contains-empty-perl-directory

Modified: trunk/testset/manpages/debian/rules
===
--- trunk/testset/manpages/debian/rules 2008-01-20 20:54:50 UTC (rev 1130)
+++ trunk/testset/manpages/debian/rules 2008-01-22 01:26:03 UTC (rev 1131)
@@ -65,6 +65,9 @@
install -m644 foo.info $(tmp)/usr/share/info/baz.broken.gz
touch $(tmp)/usr/share/info/image.png
 
+# Emit an info tag to test override handling.
+   mkdir -p $(tmp)/usr/lib/perl5
+
install -d $(tmp)/usr/share/doc/manpages
cp debian/changelog $(tmp)/usr/share/doc/manpages/changelog
gzip -9 $(tmp)/usr/share/doc/manpages/changelog

Modified: trunk/testset/tags.manpages
===
--- trunk/testset/tags.manpages 2008-01-20 20:54:50 UTC (rev 1130)
+++

lintian: r1132 - in trunk: checks debian testset

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 02:39:20 +0100 (Tue, 22 Jan 2008)
New Revision: 1132

Modified:
   trunk/checks/common_data.pm
   trunk/debian/changelog
   trunk/testset/tags.relations
Log:
* checks/common_data.pm:
  + [RA] Add Bugs, Origin, and Breaks to known binary and udeb fields
and Bugs to known source fields, matching current dpkg.  Thanks,
Rapha?\195?\171l Hertzog.  (Closes: #459787)

Modified: trunk/checks/common_data.pm
===
--- trunk/checks/common_data.pm 2008-01-22 01:26:03 UTC (rev 1131)
+++ trunk/checks/common_data.pm 2008-01-22 01:39:20 UTC (rev 1132)
@@ -87,21 +87,23 @@
  'build-conflicts', 'build-conflicts-indep', 'format', 'origin',
  'uploaders', 'python-version', 'autobuild', 'homepage', 'vcs-arch',
  'vcs-bzr', 'vcs-cvs', 'vcs-darcs', 'vcs-git', 'vcs-hg', 'vcs-mtn',
- 'vcs-svn', 'vcs-browser', 'dm-upload-allowed');
+ 'vcs-svn', 'vcs-browser', 'dm-upload-allowed', 'bugs');
 
 # The Ubuntu original-maintainer field is handled separately.
 %known_binary_fields = map { $_ = 1 }
 ('package', 'version', 'architecture', 'depends', 'pre-depends',
  'recommends', 'suggests', 'enhances', 'conflicts', 'provides',
- 'replaces', 'essential', 'maintainer', 'section', 'priority',
- 'source', 'description', 'installed-size', 'python-version', 'homepage');
+ 'replaces', 'breaks', 'essential', 'maintainer', 'section', 'priority',
+ 'source', 'description', 'installed-size', 'python-version', 'homepage',
+ 'bugs', 'origin');
 
 # The Ubuntu original-maintainer field is handled separately.
 %known_udeb_fields = map { $_ = 1 }
 ('package', 'version', 'architecture', 'subarchitecture', 'depends',
- 'recommends', 'enhances', 'provides', 'installer-menu-item',
- 'replaces', 'maintainer', 'section', 'priority',
- 'source', 'description', 'installed-size', 'kernel-version');
+ 'recommends', 'enhances', 'provides', 'replaces', 'breaks', 'replaces',
+ 'maintainer', 'section', 'priority', 'source', 'description',
+ 'installed-size', 'kernel-version', 'installer-menu-item', 'bugs',
+ 'origin');
 
 %known_obsolete_fields = map { $_ = 1 }
 ('revision', 'package-revision', 'package_revision',
@@ -183,3 +185,9 @@
 ( 'html', 'text', 'pdf', 'postscript', 'info', 'dvi', 'debiandoc-sgml' );
 
 1;
+
+# Local Variables:
+# indent-tabs-mode: nil
+# cperl-indent-level: 4
+# End:
+# vim: syntax=perl sw=4 sts=4 ts=4 et shiftround

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 01:26:03 UTC (rev 1131)
+++ trunk/debian/changelog  2008-01-22 01:39:20 UTC (rev 1132)
@@ -1,5 +1,9 @@
 lintian (1.23.43) UNRELEASED; urgency=low
 
+  * checks/common_data.pm:
++ [RA] Add Bugs, Origin, and Breaks to known binary and udeb fields
+  and Bugs to known source fields, matching current dpkg.  Thanks,
+  Raphaël Hertzog.  (Closes: #459787)
   * checks/debhelper:
 + [RA] CDBS now defaults to a debhelper compatibility level of V5.
   * checks/fields:
@@ -20,7 +24,7 @@
 + [RA] Add a spelling correction for PostgreSQL and lots of new
   capitalization corrections after a debian-devel thread.
 
- -- Russ Allbery [EMAIL PROTECTED]  Mon, 21 Jan 2008 17:25:43 -0800
+ -- Russ Allbery [EMAIL PROTECTED]  Mon, 21 Jan 2008 17:36:01 -0800
 
 lintian (1.23.42) unstable; urgency=low
 

Modified: trunk/testset/tags.relations
===
--- trunk/testset/tags.relations2008-01-22 01:26:03 UTC (rev 1131)
+++ trunk/testset/tags.relations2008-01-22 01:39:20 UTC (rev 1132)
@@ -31,10 +31,6 @@
 I: relations source: build-depends-without-arch-dep foo
 I: relations source: build-depends-without-arch-dep libc6-dev
 I: relations source: build-depends-without-arch-dep mail-transport-agent
-I: relations-multiple-libs: unknown-field-in-control bugs
-I: relations-multiple-libs: unknown-field-in-control origin
-I: relations: unknown-field-in-control bugs
-I: relations: unknown-field-in-control origin
 W: relations source: ancient-standards-version 3.1.1 (current is 3.7.3)
 W: relations source: bad-homepage lintian.debian.org
 W: relations source: package-has-a-duplicate-build-relation foo (= 3) [!amd64 
!i386], foo ( 4) [!amd64 !i386]


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

Processed: Lintian bugs fixed in revision r1132

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1132 by rra
 tag 459787 + pending
Bug#459787: lintian doesn't accept origin and bugs field in binary packages
There were no tags set.
Tags added: pending

 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]

lintian: r1133 - in trunk: checks debian testset testset/scripts testset/scripts/debian

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 05:24:55 +0100 (Tue, 22 Jan 2008)
New Revision: 1133

Added:
   trunk/testset/scripts/init-lsb-other
Modified:
   trunk/checks/init.d
   trunk/checks/init.d.desc
   trunk/debian/changelog
   trunk/testset/scripts/debian/postinst
   trunk/testset/scripts/debian/postrm
   trunk/testset/scripts/debian/rules
   trunk/testset/scripts/debian/scripts.conffiles
   trunk/testset/tags.scripts
Log:
* checks/init.d{.desc,}:
  + [RA] Downgrade a missing LSB Short-Description keyword to info since
it's not required for functionality.  Thanks, Petter Reinholdtsen.
(Closes: #460499)

Modified: trunk/checks/init.d
===
--- trunk/checks/init.d 2008-01-22 01:39:20 UTC (rev 1132)
+++ trunk/checks/init.d 2008-01-22 04:24:55 UTC (rev 1133)
@@ -204,7 +204,11 @@
} else {
for my $keyword (keys %lsb_keywords) {
if ($lsb_keywords{$keyword}  !defined $lsb{$keyword}) {
-   tag init.d-script-missing-lsb-keyword, /etc/init.d/$_ 
$keyword;
+   if ($keyword eq 'short-description') {
+   tag init.d-script-missing-lsb-short-description, 
/etc/init.d/$_;
+   } else {
+   tag init.d-script-missing-lsb-keyword, 
/etc/init.d/$_ $keyword;
+   }
}
}
}

Modified: trunk/checks/init.d.desc
===
--- trunk/checks/init.d.desc2008-01-22 01:39:20 UTC (rev 1132)
+++ trunk/checks/init.d.desc2008-01-22 04:24:55 UTC (rev 1133)
@@ -135,6 +135,14 @@
  should be empty, please still include it in the LSB keyword section with
  an empty value.
 
+Tag: init.d-script-missing-lsb-short-description
+Type: info
+Ref: http://wiki.debian.org/LSBInitScripts
+Info: This tt/etc/init.d/tt script has an LSB keyword section, but it
+ is missing a Short-Description LSB keyword.  This field isn't directly
+ used currently, but adding it is still a good idea for documentation
+ purposes.
+
 Tag: init.d-script-has-bad-start-runlevel
 Type: warning
 Ref: http://wiki.debian.org/LSBInitScripts

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 01:39:20 UTC (rev 1132)
+++ trunk/debian/changelog  2008-01-22 04:24:55 UTC (rev 1133)
@@ -12,6 +12,10 @@
   * checks/files.desc:
 + [RA] In many of the long descriptions for symlink-related tags,
   mention that running dh_link will fix symlink problems.
+  * checks/init.d{.desc,}:
++ [RA] Downgrade a missing LSB Short-Description keyword to info since
+  it's not required for functionality.  Thanks, Petter Reinholdtsen.
+  (Closes: #460499)
   * checks/menu-format:
 + [RA] Avoid a Perl warning for desktop entries without Exec.
 

Modified: trunk/testset/scripts/debian/postinst
===
--- trunk/testset/scripts/debian/postinst   2008-01-22 01:39:20 UTC (rev 
1132)
+++ trunk/testset/scripts/debian/postinst   2008-01-22 04:24:55 UTC (rev 
1133)
@@ -9,3 +9,6 @@
 if [ -x /etc/init.d/skeleton ] ; then
 update-rc.d skeleton defaults /dev/null
 fi
+if [ -x /etc/init.d/lsb-other ] ; then
+update-rc.d lsb-other defaults /dev/null
+fi

Modified: trunk/testset/scripts/debian/postrm
===
--- trunk/testset/scripts/debian/postrm 2008-01-22 01:39:20 UTC (rev 1132)
+++ trunk/testset/scripts/debian/postrm 2008-01-22 04:24:55 UTC (rev 1133)
@@ -4,4 +4,6 @@
 update-rc.d lsb-broken remove /dev/null
 update-rc.d no-lsb remove /dev/null
 update-rc.d skeleton remove /dev/null
+update-rc.d lsb-other remove /dev/null
+update-rc.d lsb-other remove /dev/null
 fi

Modified: trunk/testset/scripts/debian/rules
===
--- trunk/testset/scripts/debian/rules  2008-01-22 01:39:20 UTC (rev 1132)
+++ trunk/testset/scripts/debian/rules  2008-01-22 04:24:55 UTC (rev 1133)
@@ -50,6 +50,7 @@
install -m 755 init-skeleton $(tmp)/etc/init.d/skeleton
install -m 755 init-no-lsb $(tmp)/etc/init.d/no-lsb
install -m 755 init-lsb-broken $(tmp)/etc/init.d/lsb-broken
+   install -m 755 init-lsb-other $(tmp)/etc/init.d/lsb-other
 
install -m 755 phpfoo $(tmp)/usr/share/scripts/
sed 's/php$$/php5/' phpfoo  $(tmp)/usr/share/scripts/php5foo

Modified: trunk/testset/scripts/debian/scripts.conffiles
===
--- trunk/testset/scripts/debian/scripts.conffiles  2008-01-22 01:39:20 UTC 
(rev 1132)
+++ trunk/testset/scripts/debian/scripts.conffiles  2008-01-22 04:24:55 UTC 
(rev 1133)
@@ -1,4 +1,5 @@
 /etc/init.d/lsb-broken
+/etc/init.d/lsb-other
 /etc/init.d/no-lsb
 /etc/X11/Xsession.d/xsession-test

Fw: Переезд без головной боли

2008-01-21 Thread Craig 
   Качественный переезд

Полный комплекс услуг по офисным и квартирным переездам
   Наши цены  ИНДИВИДУАЛЬНЫ  для каждого конкретного случая
 Любая форма оплаты
   Без выходных

  772~17-10



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

Processed: Lintian bugs fixed in revision r1133

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1133 by rra
 tag 460499 + pending
Bug#460499: lintian: No need to warn about missing short-description LSB header 
in init.d scripts
There were no tags set.
Tags added: pending

 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#461978: lintian: spurious Perl warnings with --unpack

2008-01-21 Thread Russ Allbery 
Håkon Stordahl [EMAIL PROTECTED] writes:

 Package: lintian
 Version: 1.23.42
 Severity: minor
 Tags: patch

 Use of the --unpack action with Lintian causes Perl to print some
 harmless, but a bit annoying, warnings:

 $ lintian --lab /tmp/lab --setup-lab
 $ lintian --lab /tmp/lab --unpack lintian_1.23.42_all.deb 
 Use of uninitialized value in addition (+) at /usr/bin/lintian line 1427.
 Use of uninitialized value in addition (+) at /usr/bin/lintian line 1427.

I can't duplicate this.  The same commands produce no warnings for me.
Regardless, though, the code that produces those warnings shouldn't even
be running when the action isn't --check, which I'll fix.

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

lintian: r1135 - in trunk: debian frontend

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 05:45:43 +0100 (Tue, 22 Jan 2008)
New Revision: 1135

Modified:
   trunk/debian/changelog
   trunk/frontend/lintian
Log:
  + [RA] Don't try to report overrides when not checking a package.
Thanks, H?\195?\165kon Stordahl.  (Closes: #461978)

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 04:44:33 UTC (rev 1134)
+++ trunk/debian/changelog  2008-01-22 04:45:43 UTC (rev 1135)
@@ -26,6 +26,8 @@
 + [RA] Include info tags in the broken-down description of the
   override count since they're included in the total.  Based on a
   patch by Chris Lamb.  (Closes: #459851)
++ [RA] Don't try to report overrides when not checking a package.
+  Thanks, Håkon Stordahl.  (Closes: #461978)
 
   * lib/Spelling.pm:
 + [RA] Add a spelling correction for PostgreSQL and lots of new

Modified: trunk/frontend/lintian
===
--- trunk/frontend/lintian  2008-01-22 04:44:33 UTC (rev 1134)
+++ trunk/frontend/lintian  2008-01-22 04:45:43 UTC (rev 1135)
@@ -1423,10 +1423,10 @@
close(STATUS);
 }
 }
-if (not $quiet and not $no_override and not $show_overrides) {
-my $errors = $overrides{errors};
-my $warnings = $overrides{warnings};
-my $info = $overrides{info};
+if ($action eq 'check' and not $quiet and not $no_override and not 
$show_overrides) {
+my $errors = $overrides{errors} || 0;
+my $warnings = $overrides{warnings} || 0;
+my $info = $overrides{info} || 0;
 my $total = $errors + $warnings + $info;
 if ($total  0) {
my $total = ($total == 1)


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

lintian: r1136 - in trunk: checks debian

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 05:53:00 +0100 (Tue, 22 Jan 2008)
New Revision: 1136

Modified:
   trunk/checks/infofiles
   trunk/checks/menus
   trunk/debian/changelog
Log:
* checks/infofiles:
  + [RA] Don't issue unknown-intepreter for maintainer scripts with
weird interpreters.  checks/scripts already handles this and the tag
wasn't defined.  Thanks, Thijs Kinkhorst.  (Closes: #460964)
* checks/menus:
  + [RA] Don't issue unknown-interpreter for maintainer scripts with
weird interpreters.

Modified: trunk/checks/infofiles
===
--- trunk/checks/infofiles  2008-01-22 04:45:43 UTC (rev 1135)
+++ trunk/checks/infofiles  2008-01-22 04:53:00 UTC (rev 1136)
@@ -186,7 +186,6 @@
}
$interp = 'unknown';
}
-   tag unknown-interpreter, $script, $interp;
 }
 
 my $hold;

Modified: trunk/checks/menus
===
--- trunk/checks/menus  2008-01-22 04:45:43 UTC (rev 1135)
+++ trunk/checks/menus  2008-01-22 04:53:00 UTC (rev 1136)
@@ -560,7 +560,6 @@
}
$interp = 'unknown';
}
-   tag unknown-interpreter, $script, $interp;
 }
 
 while (IN) {

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 04:45:43 UTC (rev 1135)
+++ trunk/debian/changelog  2008-01-22 04:53:00 UTC (rev 1136)
@@ -15,12 +15,19 @@
   * checks/files.desc:
 + [RA] In many of the long descriptions for symlink-related tags,
   mention that running dh_link will fix symlink problems.
+  * checks/infofiles:
++ [RA] Don't issue unknown-intepreter for maintainer scripts with
+  weird interpreters.  checks/scripts already handles this and the tag
+  wasn't defined.  Thanks, Thijs Kinkhorst.  (Closes: #460964)
   * checks/init.d{.desc,}:
 + [RA] Downgrade a missing LSB Short-Description keyword to info since
   it's not required for functionality.  Thanks, Petter Reinholdtsen.
   (Closes: #460499)
   * checks/menu-format:
 + [RA] Avoid a Perl warning for desktop entries without Exec.
+  * checks/menus:
++ [RA] Don't issue unknown-interpreter for maintainer scripts with
+  weird interpreters.
 
   * frontend/lintian:
 + [RA] Include info tags in the broken-down description of the


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

lintian: r1134 - in trunk: checks debian testset/cdbs-test/debian

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 05:44:33 +0100 (Tue, 22 Jan 2008)
New Revision: 1134

Modified:
   trunk/checks/changelog-file
   trunk/debian/changelog
   trunk/testset/cdbs-test/debian/changelog
Log:
* checks/changelog-file:
  + [RA] Decode the changelog entry from UTF-8 before doing length
checks.  Thanks, Cyril Brulebois.  (Closes: #461822)

Modified: trunk/checks/changelog-file
===
--- trunk/checks/changelog-file 2008-01-22 04:24:55 UTC (rev 1133)
+++ trunk/checks/changelog-file 2008-01-22 04:44:33 UTC (rev 1134)
@@ -23,6 +23,8 @@
 use Spelling;
 use Tags;
 use Util;
+
+use Encode qw(decode);
 use Parse::DebianChangelog;
 
 sub run {
@@ -316,7 +318,12 @@
 while ($changes =~ /(closes\s*(?:bug)?\#?\s?\d{6,})[^\w]/ig) {
tag possible-missing-colon-in-closes, $1 if $1;
 }
-my @lines = split ('\n', $changes);
+
+# We have to decode into UTF-8 to get the right length for the length
+# check.  For some reason, use open ':utf8' isn't sufficient.  If the
+# changelog uses a non-UTF-8 encoding, this will mangle it, but it doesn't
+# matter for the length check.
+my @lines = split ('\n', decode ('utf-8', $changes));
 for my $i (0 .. $#lines) {
 if (length ($lines[$i])  80  $lines[$i] !~ /^[\s.o*+-]*\S+$/) {
 tag 'debian-changelog-line-too-long', line  . ($i + 1);

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 04:24:55 UTC (rev 1133)
+++ trunk/debian/changelog  2008-01-22 04:44:33 UTC (rev 1134)
@@ -1,5 +1,8 @@
 lintian (1.23.43) UNRELEASED; urgency=low
 
+  * checks/changelog-file:
++ [RA] Decode the changelog entry from UTF-8 before doing length
+  checks.  Thanks, Cyril Brulebois.  (Closes: #461822)
   * checks/common_data.pm:
 + [RA] Add Bugs, Origin, and Breaks to known binary and udeb fields
   and Bugs to known source fields, matching current dpkg.  Thanks,

Modified: trunk/testset/cdbs-test/debian/changelog
===
--- trunk/testset/cdbs-test/debian/changelog2008-01-22 04:24:55 UTC (rev 
1133)
+++ trunk/testset/cdbs-test/debian/changelog2008-01-22 04:44:33 UTC (rev 
1134)
@@ -3,7 +3,8 @@
   * Backport, but for some reason I go on and on for longer than 80 columns 
thus making the entry annoying to read.
   * 
this-is-a-really-long-dpatch-name-that-contains-the-entire-bug-report-in-the-file-name.dpatch:
 - New.
-
+  * Not really too long: “héh锓héh锓héh锓héh锓héh锓héhé”
+  
  -- Russ Allbery [EMAIL PROTECTED]  Wed, 05 Dec 2007 17:23:59 -0800
 
 cdbs-test (1-1) unstable; urgency=low


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

Processed: Lintian bugs fixed in revision r1135

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1135 by rra
 tag 461978 + pending
Bug#461978: lintian: spurious Perl warnings with --unpack
Tags were: patch
Tags added: pending

 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]

Processed: Lintian bugs fixed in revision r1134

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1134 by rra
 tag 461822 + pending
Bug#461822: lintian: Please consider correct (Unicode-aware) line length in 
debian/changelog for line-too-long
There were no tags set.
Tags added: pending

 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]

Processed: Lintian bugs fixed in revision r1137

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1137 by rra
 tag 461982 + pending
Bug#461982: lintian: breaks when laboratory directory is specified as a 
relative path
Tags were: patch
Tags added: pending

 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]

lintian: r1138 - in trunk: checks debian testset testset/manpages

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 06:12:11 +0100 (Tue, 22 Jan 2008)
New Revision: 1138

Modified:
   trunk/checks/manpages
   trunk/debian/changelog
   trunk/testset/manpages/include.3
   trunk/testset/tags.manpages
Log:
* checks/manpages:
  + [RA] Make hyphen-used-as-minus-sign more conservative to avoid false
positives with non-ASCII text and catch hyphens at the start of
lines.  Thanks, Michal ?\196?\140iha?\197?\153.  (Closes: #459871)

Modified: trunk/checks/manpages
===
--- trunk/checks/manpages   2008-01-22 04:56:47 UTC (rev 1137)
+++ trunk/checks/manpages   2008-01-22 05:12:11 UTC (rev 1138)
@@ -304,9 +304,11 @@
# Catch hyphens used as minus signs by looking for ones at the
# beginning of a word, but don't generate false positives on \s-1
# (small font), \*(-- (pod2man long dash), or things like \h'-1'.
-   if ($line =~ /^[^\.].*
-  [^\w\\]
-  (?! \\s | \*\( | \(- | \w\' )
+   if ($line =~ /^(
+([^\.].*)?
+[\s\'\\`\(\[]
+(?! \\s | \*\( | \(- | \w\' )
+   )?
   (--?\w+)/ox) {
$hc++;
tag hyphen-used-as-minus-sign, $file:$lc if $hc = 10 or 
$ENV{'LINTIAN_DEBUG'};

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 04:56:47 UTC (rev 1137)
+++ trunk/debian/changelog  2008-01-22 05:12:11 UTC (rev 1138)
@@ -23,6 +23,10 @@
 + [RA] Downgrade a missing LSB Short-Description keyword to info since
   it's not required for functionality.  Thanks, Petter Reinholdtsen.
   (Closes: #460499)
+  * checks/manpages:
++ [RA] Make hyphen-used-as-minus-sign more conservative to avoid false
+  positives with non-ASCII text and catch hyphens at the start of
+  lines.  Thanks, Michal Čihař.  (Closes: #459871)
   * checks/menu-format:
 + [RA] Avoid a Perl warning for desktop entries without Exec.
   * checks/menus:

Modified: trunk/testset/manpages/include.3
===
--- trunk/testset/manpages/include.32008-01-22 04:56:47 UTC (rev 1137)
+++ trunk/testset/manpages/include.32008-01-22 05:12:11 UTC (rev 1138)
@@ -4,5 +4,13 @@
 .SH INCLUDED BIT
 .nr zY 1
 .so man3/included.3
+.SH HYPHENS
+This isn't a hyphen\*(--just a long dash\*(--and \h'-1' also isn't, nor
+should we warn about `\-' or \-a, but -a
+and `-' (nope, that's fine) are bad news.  So is
+--foo and
+(--bar)
+--baz
+and '--foo'.
 .SH SEE ALSO
 included(3)

Modified: trunk/testset/tags.manpages
===
--- trunk/testset/tags.manpages 2008-01-22 04:56:47 UTC (rev 1137)
+++ trunk/testset/tags.manpages 2008-01-22 05:12:11 UTC (rev 1138)
@@ -32,6 +32,11 @@
 E: manpages: stray-directory-in-manpage-directory 
usr/share/man/man1/not-a-man-page.1.gz/
 E: manpages: subdir-in-usr-bin usr/bin/bin/
 E: manpages: x11-games-should-be-in-usr-games 
usr/X11R6/man/man6/X11R6-binary.man
+I: manpages: hyphen-used-as-minus-sign usr/share/man/man3/include.3.gz:11
+I: manpages: hyphen-used-as-minus-sign usr/share/man/man3/include.3.gz:12
+I: manpages: hyphen-used-as-minus-sign usr/share/man/man3/include.3.gz:13
+I: manpages: hyphen-used-as-minus-sign usr/share/man/man3/include.3.gz:14
+I: manpages: hyphen-used-as-minus-sign usr/share/man/man3/include.3.gz:9
 I: manpages: no-md5sums-control-file
 I: manpages: unused-override foo-tag-that-does-not-exist
 N: 2 tags overridden (1 error, 1 info)


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

Bug#457413: marked as done (Check section header in source package)

2008-01-21 Thread Debian Bug Tracking System 
Your message dated Mon, 21 Jan 2008 21:15:15 -0800
with message-id [EMAIL PROTECTED]
and subject line Bug#457413: Check section header in source package
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

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

---BeginMessage---
Package: lintian
Version: 1.23.41
Severity: wishlist

Hello,

in one of my new packages I accidentially missed the section: header
in the control file which caused ftpmaster to reject the package.
I would regard it reasonable if lintian would check that source and
binary contain a section: header.

Kind regards and thanks for maintaining lintian

   Andreas.

-- System Information:
Debian Release: lenny/sid
  APT prefers testing
  APT policy: (501, 'testing')
Architecture: i386 (i686)

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

Versions of packages lintian depends on:
ii  binutils2.18.1~cvs20071027-1 The GNU assembler, linker and bina
ii  diffstat1.45-2   produces graph of changes introduc
ii  dpkg-dev1.14.7   package building tools for Debian
ii  file4.21-3   Determines file type using magic
ii  gettext 0.16.1-2 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  liburi-perl 1.35.dfsg.1-1Manipulates and accesses URI strin
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


---End Message---
---BeginMessage---
Lintian already does this check.  Please re-open with more details if I
didn't understand what you were asking for.

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

---End Message---

Processed: Lintian bugs fixed in revision r1139

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1139 by rra
 tag 458164 + pending
Bug#458164: Lintian should not suggest the Makefile test for packages that use 
a static Makefile
There were no tags set.
Tags added: pending

 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]

Processed: Lintian bugs fixed in revision r1138

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1138 by rra
 tag 459871 + pending
Bug#459871: hyphen-used-as-minus-sign false positives
Tags were: patch
Tags added: pending

 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]

lintian: r1139 - in trunk: checks debian

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 06:14:18 +0100 (Tue, 22 Jan 2008)
New Revision: 1139

Modified:
   trunk/checks/rules.desc
   trunk/debian/changelog
Log:
* checks/rules.desc:
  + [RA] To fix an ignores-make-clean-error, suggest removing - for
static makefiles.  Thanks, Andrea Colangelo.  (Closes: #458164)

Modified: trunk/checks/rules.desc
===
--- trunk/checks/rules.desc 2008-01-22 05:12:11 UTC (rev 1138)
+++ trunk/checks/rules.desc 2008-01-22 05:14:18 UTC (rev 1139)
@@ -52,7 +52,7 @@
  .
   -$(MAKE) distclean
  .
- The leading - tells make to ignore all errors.  Normally this is done
+ The leading - tells make to ignore all errors.  Normally this is done
  for packages using Autoconf since Makefile may not exist.  However, this
  line ignores all other error messages, not just the missing Makefile
  error.  It's better to use:
@@ -60,7 +60,7 @@
   [ ! -f Makefile ] || $(MAKE) distclean
  .
  so that other error messages from the clean or distclean rule will still
- be caught.
+ be caught (or just remove the - if the package uses a static makefile).
 
 Tag: debian-rules-sets-DH_COMPAT
 Type: warning

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 05:12:11 UTC (rev 1138)
+++ trunk/debian/changelog  2008-01-22 05:14:18 UTC (rev 1139)
@@ -32,6 +32,9 @@
   * checks/menus:
 + [RA] Don't issue unknown-interpreter for maintainer scripts with
   weird interpreters.
+  * checks/rules.desc:
++ [RA] To fix an ignores-make-clean-error, suggest removing - for
+  static makefiles.  Thanks, Andrea Colangelo.  (Closes: #458164)
 
   * frontend/lintian:
 + [RA] Include info tags in the broken-down description of the


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

lintian: r1137 - in trunk: debian frontend

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 05:56:47 +0100 (Tue, 22 Jan 2008)
New Revision: 1137

Modified:
   trunk/debian/changelog
   trunk/frontend/lintian
Log:
  + [RA] Cope with a relative path to the lab.  Patch from H?\195?\165kon
Stordahl.  (Closes: #461982)

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 04:53:00 UTC (rev 1136)
+++ trunk/debian/changelog  2008-01-22 04:56:47 UTC (rev 1137)
@@ -35,6 +35,8 @@
   patch by Chris Lamb.  (Closes: #459851)
 + [RA] Don't try to report overrides when not checking a package.
   Thanks, Håkon Stordahl.  (Closes: #461978)
++ [RA] Cope with a relative path to the lab.  Patch from Håkon
+  Stordahl.  (Closes: #461982)
 
   * lib/Spelling.pm:
 + [RA] Add a spelling correction for PostgreSQL and lots of new

Modified: trunk/frontend/lintian
===
--- trunk/frontend/lintian  2008-01-22 04:53:00 UTC (rev 1136)
+++ trunk/frontend/lintian  2008-01-22 04:56:47 UTC (rev 1137)
@@ -1160,6 +1160,9 @@
 
 # determine base directory
 my $base = $LINTIAN_LAB/$long_type/$pkg;
+unless ($base =~ m,^/,) {
+   $base = $cwd/$base;
+}
 print N: Base directory in lab: $base\n if $debug;
 
 my $act_unpack_level = 0;


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

lintian: r1141 - in trunk: debian lib

2008-01-21 Thread rra 
Author: rra
Date: 2008-01-22 06:26:31 +0100 (Tue, 22 Jan 2008)
New Revision: 1141

Modified:
   trunk/debian/changelog
   trunk/lib/Spelling.pm
Log:
  + [RA] Add spelling corrections for PostgreSQL and OCaml and lots of
new capitalization corrections.  (Closes: #460347)

Modified: trunk/debian/changelog
===
--- trunk/debian/changelog  2008-01-22 05:24:02 UTC (rev 1140)
+++ trunk/debian/changelog  2008-01-22 05:26:31 UTC (rev 1141)
@@ -52,8 +52,8 @@
   Stordahl.  (Closes: #461982)
 
   * lib/Spelling.pm:
-+ [RA] Add a spelling correction for PostgreSQL and lots of new
-  capitalization corrections after a debian-devel thread.
++ [RA] Add spelling corrections for PostgreSQL and OCaml and lots of
+  new capitalization corrections.  (Closes: #460347)
 
  -- Russ Allbery [EMAIL PROTECTED]  Mon, 21 Jan 2008 17:36:01 -0800
 

Modified: trunk/lib/Spelling.pm
===
--- trunk/lib/Spelling.pm   2008-01-22 05:24:02 UTC (rev 1140)
+++ trunk/lib/Spelling.pm   2008-01-22 05:26:31 UTC (rev 1141)
@@ -106,6 +106,7 @@
   convertor converter
   convinient convenient
   cryptocraphic cryptographic
+  dbus D-Bus
   deamon daemon
   debain Debian
   debians Debian\'s
@@ -212,6 +213,7 @@
   necesary necessary
   nescessary necessary
   noticable noticeable
+  o\'caml OCaml
   optionnal optional
   orientatied orientated
   orientied oriented
@@ -317,11 +319,8 @@
 # descriptions, and should not be applied to files that may contain
 # configuration fragments or more informal files such as debian/copyright.
 our %CORRECTIONS_CASE = qw(
-   DBUS D-Bus
-   DBus D-Bus
D-BUS D-Bus
d-bus D-Bus
-   dbus D-Bus
debian Debian
english English
french French
@@ -349,7 +348,7 @@
OCAML OCaml
Ocaml OCaml
ocaml OCaml
-   Postgresql PostgreSQL   
+   Postgresql PostgreSQL
postgresql PostgreSQL
python Python
russian Russian


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

Processed: Lintian bugs fixed in revision r1141

2008-01-21 Thread Debian Bug Tracking System 
Processing commands for [EMAIL PROTECTED]:

 package lintian
Ignoring bugs not assigned to: lintian

 # Fixed in r1141 by rra
 tag 460347 + pending
Bug#460347: lintian: correct capitalization of projet names: OCaml
Tags were: patch
Tags added: pending

 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]