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

Bug#575661: lintian: Fix to col command error



Package: lintian
Version: 2.3.4
Severity: normal
Tags: patch

Since this test should work in all environment, I think whoever created
this test must have chosen LANG=C with reason.  I am not even sure if
en_US.UTF-8 exists in the test environment.

Then I also realized there are manpages in many old encodings and they
need to be tested by this script.  Testing for the good UTF-8-ness is
not the right choice (although I suggested so in the previous report.)

So the right thing to do is like other exceptions given in the code.

Please apply attached patch to /usr/share/lintian/checks/manpages
(of course in source file as lintian-2.3.4/checks/manpages)

Osamu

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

Kernel: Linux 2.6.32-3-amd64 (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.20.1-5          The GNU assembler, linker and bina
ii  diffstat               1.47-1            produces graph of changes introduc
ii  dpkg-dev               1.15.5.6          Debian package development tools
ii  file                   5.04-2            Determines file type using "magic"
ii  gettext                0.17-10           GNU Internationalization utilities
ii  intltool-debian        0.35.0+20060710.1 Help i18n of RFC822 compliant conf
ii  libapt-pkg-perl        0.1.24            Perl interface to libapt-pkg
ii  libclass-accessor-perl 0.34-1            Perl module that automatically gen
ii  libipc-run-perl        0.89-1            Perl module for running processes
ii  libparse-debianchangel 1.1.1-2           parse Debian changelogs and output
ii  libtimedate-perl       1.2000-1          collection of modules to manipulat
ii  liburi-perl            1.54-1            module to manipulate and access UR
ii  locales-all [locales]  2.10.2-6          Embedded GNU C Library: Precompile
ii  man-db                 2.5.7-2           on-line manual pager
ii  perl [libdigest-sha-pe 5.10.1-11         Larry Wall's Practical Extraction 

lintian recommends no packages.

Versions of packages lintian suggests:
pn  binutils-multiarch            <none>     (no description available)
pn  libtext-template-perl         <none>     (no description available)
ii  man-db                        2.5.7-2    on-line manual pager

-- no debconf information

-- debsums errors found:
debsums: changed file /usr/share/lintian/checks/manpages (from lintian package)i

Yes, I tested my patch :-)
I overrote this file with the fixed file (see attached patch)
--- manpages.orig	2010-04-10 15:30:41.000000000 +0900
+++ manpages	2010-04-10 15:41:44.000000000 +0900
@@ -258,6 +258,9 @@
 	    next if /^\s*$/;
 	    # ignore errors from gzip, will be dealt with at other places
 	    next if /^(man|gzip)/;
+	    # ignore errors from col since we are forcing LANG=C which is 
+	    # known to be incompatible for UTF-8 output generated by "-E UTF-8".
+	    next if /^col: Invalid or incomplete multibyte or wide character/;
 	    # ignore wrapping failures for Asian man pages (groff problem)
 	    if ($lang =~ /^(?:ja|ko|zh)/) {
 		next if /warning \[.*\]: cannot adjust line/;

Reply to: