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

[SCM] Debian package checker branch, master, updated. 2.2.18-87-g7ce5406



The following commit has been merged in the master branch:
commit 7ce5406c0af3836cd22e4087ddeb36d8ec5bdcec
Author: Raphael Geissert <atomo64@gmail.com>
Date:   Fri Dec 25 20:59:21 2009 -0600

    Check for spelling errors in English manpages
    
    * checks/manpages{,.desc}:
      + [RG] Check for spelling errors in English manpages.

diff --git a/checks/manpages b/checks/manpages
index 3e10af5..c0cce98 100644
--- a/checks/manpages
+++ b/checks/manpages
@@ -22,6 +22,7 @@ package Lintian::manpages;
 use strict;
 use Util;
 use Lintian::Tags qw(tag);
+use Spelling;
 
 sub run {
 
@@ -315,6 +316,9 @@ foreach my $file (sort keys %{$info->index}) {
 	    if ($line eq '.SH "POD ERRORS"') {
 		tag "manpage-has-errors-from-pod2man", "$file:$lc";
 	    }
+	    # Check for spelling errors if the manpage is English
+	    spelling_check("manpage-has-spelling-error", $line, $file)
+		if ($path =~ m,/man/man\d/,);
 	}
 	tag "hyphen-used-as-minus-sign", $file, ($hc - 10),
 	    "more occurrences not shown"
diff --git a/checks/manpages.desc b/checks/manpages.desc
index 479949c..61567ae 100644
--- a/checks/manpages.desc
+++ b/checks/manpages.desc
@@ -280,3 +280,17 @@ Info: This package installs a manual page in a locale directory that
  language.  The known cases where country names are appropriate are pt_BR
  and zh_*.  Please file a bug against Lintian if this is another case
  where a country name is appropriate.
+
+Tag: manpage-has-spelling-error
+Severity: normal
+Certainty: possible
+Info: Lintian found a spelling error in the manpage. Lintian has a list
+ of common misspellings that it looks for.  It does not have a
+ dictionary like a spelling checker does.
+ .
+ If the string containing the spelling error is translated with the help
+ of gettext (with the help of po4a, for example) or a similar tool,
+ please fix the error in the translations as well as the English text to
+ avoid making the translations fuzzy.  With gettext, for example, this
+ means you should also fix the spelling mistake in the corresponding
+ msgids in the *.po files.
diff --git a/debian/changelog b/debian/changelog
index 70f1245..f8b9bd9 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -70,6 +70,8 @@ lintian (2.3.0) UNRELEASED; urgency=low
     + [RA] Add checks for missing dependencies on $local_fs or $remote_fs
       based on what files appear to be used in the init scripts.  Patch
       from Raphael Geissert.  (Partly addresses #405454)
+  * checks/manpages{,.desc}:
+    + [RG] Check for spelling errors in English manpages.
   * checks/manpages.desc:
     + [RA] Clarify the long description of hyphen-used-as-minus-sign.
       Note that Debian groff is currently patched to avoid this problem,
diff --git a/t/tests/manpages-general/debian/program.1 b/t/tests/manpages-general/debian/program.1
index 7464631..cca9c1f 100644
--- a/t/tests/manpages-general/debian/program.1
+++ b/t/tests/manpages-general/debian/program.1
@@ -39,6 +39,7 @@ These programs follow the usual GNU command line syntax, with long
 options starting with two dashes (`-').
 A summary of options is included below.
 For a complete description, see the Info files.
+This is a speling mistake.
 .TP
 .B \-h, \-\-help
 Show summary of options.
diff --git a/t/tests/manpages-general/desc b/t/tests/manpages-general/desc
index 48943c9..7f4e6b5 100644
--- a/t/tests/manpages-general/desc
+++ b/t/tests/manpages-general/desc
@@ -11,6 +11,7 @@ Test-For:
  manpage-has-bad-whatis-entry
  manpage-has-errors-from-man
  manpage-has-errors-from-pod2man
+ manpage-has-spelling-error
  manpage-has-useless-whatis-entry
  manpage-has-wrong-extension
  manpage-is-dh_make-template
diff --git a/t/tests/manpages-general/tags b/t/tests/manpages-general/tags
index fb3c3f2..580670a 100644
--- a/t/tests/manpages-general/tags
+++ b/t/tests/manpages-general/tags
@@ -41,6 +41,8 @@ W: manpages-general: manpage-has-bad-whatis-entry usr/share/man/man6/usr-games-b
 W: manpages-general: manpage-has-errors-from-man usr/share/man/man1/test.1p.gz 12: warning: macro `dep' not defined (possibly missing space after `de')
 W: manpages-general: manpage-has-errors-from-man usr/share/man/man3/included.3.gz 6: warning: macro `zY' not defined
 W: manpages-general: manpage-has-errors-from-pod2man usr/share/man/man1/test.1p.gz:17
+W: manpages-general: manpage-has-spelling-error usr/X11R6/man/man1/rstartd.1x.gz speling spelling
+W: manpages-general: manpage-has-spelling-error usr/share/man/man1/program.1.gz speling spelling
 W: manpages-general: manpage-has-useless-whatis-entry usr/share/man/man1/true.1.gz
 W: manpages-general: manpage-locale-dir-country-specific usr/share/man/de_DE/man1/test-utf8-chars.1p.gz
 W: manpages-general: manpage-section-mismatch usr/X11R6/man/man1/rstartd.1x.gz:5 1x != 1

-- 
Debian package checker


Reply to: