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

[SCM] Debian package checker branch, master, updated. 1.24.4-110-g461090e



The following commit has been merged in the master branch:
commit b73a20fbe5ccf886c7e87fa6bfa5b5f403fc8af1
Author: Frank Lichtenheld <djpig@debian.org>
Date:   Fri Sep 12 01:09:20 2008 +0200

    checks/manpages: Add new tag manpage-has-errors-from-pod2man
    
    pod2man helpfully points out errors in generated manpages in the
    manpages themself. Warn about the existance of such a section.
    
    Suggested by Enrico Zini.
    
    Closes: #497864

diff --git a/checks/manpages b/checks/manpages
index 4cd9d7c..ce15f0e 100644
--- a/checks/manpages
+++ b/checks/manpages
@@ -330,6 +330,9 @@ foreach my $file (sort keys %{$info->index}) {
 		# regexes taken from checks/files
 		tag "FSSTND-dir-in-manual-page", "$file:$lc $1";
 	    }
+	    if ($line =~ m/^.SH "POD ERRORS"$/) {
+		tag "manpage-has-errors-from-pod2man", "$file:$lc";
+	    }
 	}
 	tag "hyphen-used-as-minus-sign", $file, ($hc-10), "more occurrences not shown" if $hc > 10 and ! $ENV{'LINTIAN_DEBUG'};
     }
diff --git a/checks/manpages.desc b/checks/manpages.desc
index d9b9d30..b8097f3 100644
--- a/checks/manpages.desc
+++ b/checks/manpages.desc
@@ -184,6 +184,14 @@ Info: This man page provokes warnings or errors from man.
  To test this for yourself you can use the following command:
   LANG=C man --warnings -l manpage-file &gt;/dev/null
 
+Tag: manpage-has-errors-from-pod2man
+Type: warning
+Severity: normal
+Certainty: certain
+Info: This man page contains a section "POD ERRORS" generated by pod2man.
+ This sections lists errors in the POD syntax found by pod2man during the
+ generation of the man page.
+
 Tag: manpage-for-x11-binary-in-wrong-directory
 Type: error
 Severity: important
diff --git a/debian/changelog b/debian/changelog
index 0c21b2f..b75adf2 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -16,6 +16,7 @@ lintian (2.0.0~rc1) experimental; urgency=low
       - improbable-bug-number-in-closes
       - maintainer-also-in-uploaders
       - maintainer-script-ignores-errors
+      - manpage-has-errors-from-pod2man
       - ored-build-depends-on-obsolete-package (actually a split of
          build-depends-on-obsolete-package in two tags)
     + Removed:
@@ -57,6 +58,9 @@ lintian (2.0.0~rc1) experimental; urgency=low
       exclusive list.  Also fix a spelling mistake.
     + [ADB] Re-add an accidentally removed slash to the check for embedded
       Javascript libraries, removing some false positives.  (Closes: #497215)
+  * checks/manpages{,.desc}:
+    + [FL] Warn about the POD ERRORS sections pod2man generates.
+      (Closes: #497864)
   * checks/menus{,.desc}:
     + [FL] Do not complain if the unknown doc-base section is the subsection
       of a known section.  The doc-base documentation seems to suggest that
diff --git a/testset/manpages/test.1p b/testset/manpages/test.1p
index 4f2546d..549d64a 100644
--- a/testset/manpages/test.1p
+++ b/testset/manpages/test.1p
@@ -13,3 +13,10 @@ Here's a serious syntax error.
 .PP
 .nh
 http://example.org/really/long/url/that/cannot/be/wrapped/and/will/produce/errors/from/man
+.\" Copied from caff(1)
+.SH "POD ERRORS"
+.IX Header "POD ERRORS"
+Hey! \fBThe above document had some coding errors, which are explained below:\fR
+.IP "Around line 138:" 4
+.IX Item "Around line 138:"
+You forgot a '=back' before '=head2'
diff --git a/testset/tags.manpages b/testset/tags.manpages
index a07e173..02d53d3 100644
--- a/testset/tags.manpages
+++ b/testset/tags.manpages
@@ -65,5 +65,6 @@ W: manpages: manpage-has-bad-whatis-entry usr/share/man/man1/usr-sbin-binary.1.g
 W: manpages: manpage-has-bad-whatis-entry usr/share/man/man6/usr-games-binary.6.gz
 W: manpages: manpage-has-errors-from-man usr/share/man/man1/test.1p.gz 12: warning: `dep' not defined (probable missing space after `de')
 W: manpages: manpage-has-errors-from-man usr/share/man/man3/included.3.gz 6: warning: `zY' not defined
+W: manpages: manpage-has-errors-from-pod2man usr/share/man/man1/test.1p.gz:17
 W: manpages: manpage-has-useless-whatis-entry usr/share/man/man1/true.1.gz
 W: manpages: manpage-locale-dir-country-specific usr/share/man/de_DE/man1/test-utf8-chars.1p.gz

-- 
Debian package checker


Reply to: