[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#682701: lintian: manpage wrapping warnings should be ignored if the line contains file://



Package: lintian
Version: 2.5.10~bpo60+1
Severity: wishlist
Tags: patch

This is a followup to #491578 (http://bugs.debian.org/491578).

I think it is reasonable to add file:// to the list of url-type things
that should not trigger a wrapping warning. Patch below. Credits to
nthykier on #debian-mentors on OFTC for the fix.

                                                       Regards, Faheem

$ git diff -- checks/manpages
diff --git a/checks/manpages b/checks/manpages
index c4320cc..4421857 100644
--- a/checks/manpages
+++ b/checks/manpages
@@ -281,7 +281,7 @@ foreach my $file ($info->sorted_index) {
             # ignore wrapping failures if they contain URLs (.UE is an
             # extension for marking the end of a URL).
             next if /:(\d+): warning \[.*\]: (?:can\'t break|cannot adjust) line/
-                and (   $manfile[$1 - 1] =~ m,(?:https?|ftp)://.+,i
+                and (   $manfile[$1 - 1] =~ m,(?:https?|ftp|file)://.+,i
                      or $manfile[$1 - 1] =~ m,^\s*\.\s*UE\b,);
             # ignore common undefined macros from pod2man << Perl 5.10
             next if /warning: (?:macro )?\`(?:Tr|IX)\' not defined/;

-- System Information:
Debian Release: 6.0.5
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable'), (50, 'unstable'), (50, 'testing'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-vserver-686-bigmem (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages lintian depends on:
ii  binutils                       2.22-2
ii  bzip2                          1.0.5-6+squeeze1
ii  diffstat                       1.53-1
ii  file                           5.04-5+squeeze2
ii  gettext                        0.18.1.1-3
ii  hardening-includes             2.2~bpo60+1
ii  intltool-debian                0.35.0+20060710.1
ii  libapt-pkg-perl                0.1.24+b1
ii  libarchive-zip-perl            1.30-3
ii  libc-bin                       2.11.3-3
ii  libclass-accessor-perl         0.34-1
ii  libclone-perl                  0.31-1
ii  libdpkg-perl                   1.16.1.2~bpo60+1
ii  libemail-valid-perl            0.184-1
ii  libipc-run-perl                0.89-1
ii  libparse-debianchangelog-perl  1.1.1-2.1
ii  libtimedate-perl               1.2000-1
ii  liburi-perl                    1.54-2
ii  locales                        2.11.3-3
ii  man-db                         2.5.7-8
ii  patchutils                     0.3.1-2
ii  perl [libdigest-sha-perl]      5.10.1-17squeeze3

lintian recommends no packages.

Versions of packages lintian suggests:
pn  binutils-multiarch     <none>
ii  dpkg-dev               1.16.1.2~bpo60+1
ii  libhtml-parser-perl    3.66-1
pn  libperlio-gzip-perl    <none>
ii  libtext-template-perl  1.45-1
ii  lzma                   4.43-14
ii  man-db                 2.5.7-8
ii  xz-utils               5.0.0-2

-- no debconf information


Reply to: