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

[SCM] Debian package checker branch, master, updated. 1.24.2-14-g941101c



The following commit has been merged in the master branch:
commit cfa5341073015b3d4f95a52bc61cfc6080869fdf
Author: Russ Allbery <rra@debian.org>
Date:   Sun Jul 20 22:43:39 2008 -0700

    Supress line break warnings in man pages with URLs
    
    * checks/manpages:
      + [RA] Suppress warnings about inability to break a line that contains
        a URL.  URLs often can't be usefully broken.  Based on a patch by
        Damyan Ivanov.  (Closes: #491578)

diff --git a/checks/manpages b/checks/manpages
index ce0d4d0..24253c9 100644
--- a/checks/manpages
+++ b/checks/manpages
@@ -304,6 +304,9 @@ while (<IN>) {
 		next if /warning \[.*\]: cannot adjust line/;
 		next if /warning \[.*\]: can\'t break line/;
 	    }
+	    # ignore wrapping failures if they contain URLs
+	    next if /:(\d+): warning \[.*\]: can\'t break line/
+		and $manfile[$1 - 1] =~ m,(?:http|ftp)://.+,i;
 	    # ignore charset issues with old versions of man for all man pages
 	    # since we can't know, with old versions, whether that was just a
 	    # Unicode issue
diff --git a/debian/changelog b/debian/changelog
index 44ad4a8..7522f7f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -19,6 +19,10 @@ lintian (1.24.3) unstable; urgency=low
   * checks/fields{,.desc}:
     + [RA] Warn about packages that depend exclusively on makedev.
       Thanks, Josh Triplett.  (Closes: #491625)
+  * checks/manpages:
+    + [RA] Suppress warnings about inability to break a line that contains
+      a URL.  URLs often can't be usefully broken.  Based on a patch by
+      Damyan Ivanov.  (Closes: #491578)
   * checks/patch-systems:
     + [RA] Combine all patch-system-but-direct-changes-in-diff instances
       into a single message to reduce the noise from packages that rebuild
diff --git a/testset/manpages/test.1p b/testset/manpages/test.1p
index b87cdd7..4f2546d 100644
--- a/testset/manpages/test.1p
+++ b/testset/manpages/test.1p
@@ -10,3 +10,6 @@ Use more up-level tools like human brains to understand lintian.
 Here's a bad undefined macro.
 Here's a serious syntax error.
 .dep
+.PP
+.nh
+http://example.org/really/long/url/that/cannot/be/wrapped/and/will/produce/errors/from/man

-- 
Debian package checker


Reply to: