Author: rra
Date: 2008-01-03 05:39:27 +0100 (Thu, 03 Jan 2008)
New Revision: 1093

Modified:
   trunk/checks/changelog-file
   trunk/checks/changelog-file.desc
   trunk/debian/changelog
   trunk/testset/manpages/debian/changelog
Log:
* checks/changelog-file{.desc,}:
  + [RA] Don't spell-check lines that include the word "spelling".
    Thanks, Andreas Hoenen.  (Closes: #456515)

Modified: trunk/checks/changelog-file
===================================================================
--- trunk/checks/changelog-file 2008-01-03 04:30:43 UTC (rev 1092)
+++ trunk/checks/changelog-file 2008-01-03 04:39:27 UTC (rev 1093)
@@ -316,13 +316,17 @@
     while ($changes =~ /(closes\s*(?:bug)?\#?\s?\d{6,})[^\w]/ig) {
        tag "possible-missing-colon-in-closes", "$1" if $1;
     }
-    spelling_check('spelling-error-in-changelog', $changes);
     my @lines = split ('\n', $changes);
     for my $i (0 .. $#lines) {
         if (length ($lines[$i]) > 80 && $lines[$i] =~ /\s/) {
             tag 'debian-changelog-line-too-long', "line " . ($i + 1);
         }
     }
+
+    # Strip out all lines that contain the word spelling to avoid false
+    # positives on changelog entries for spelling fixes.
+    $changes =~ s/^.*spelling.*\n//gm;
+    spelling_check('spelling-error-in-changelog', $changes);
 }
 
 # read the changelog itself

Modified: trunk/checks/changelog-file.desc
===================================================================
--- trunk/checks/changelog-file.desc    2008-01-03 04:30:43 UTC (rev 1092)
+++ trunk/checks/changelog-file.desc    2008-01-03 04:39:27 UTC (rev 1093)
@@ -199,6 +199,10 @@
 Info: Lintian found a spelling error in the latest entry of the Debian
  changelog.  Lintian has a list of common misspellings that it looks for.
  It does not have a dictionary like a spelling checker does.
+ .
+ When writing a changelog entry for a spelling fix that includes the
+ misspelling, ensure the word "spelling" is on the same line as the
+ misspelled word to avoid triggering this warning.
 
 Tag: spelling-error-in-news-debian
 Type: warning

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-01-03 04:30:43 UTC (rev 1092)
+++ trunk/debian/changelog      2008-01-03 04:39:27 UTC (rev 1093)
@@ -1,5 +1,8 @@
 lintian (1.23.42) UNRELEASED; urgency=low
 
+  * checks/changelog-file{.desc,}:
+    + [RA] Don't spell-check lines that include the word "spelling".
+      Thanks, Andreas Hoenen.  (Closes: #456515)
   * checks/control-file{.desc,}:
     + [RA] Include the package name in stronger-dependency-implies-weaker.
     + [RA] Fix stronger-dependency-implies-weaker description cut and

Modified: trunk/testset/manpages/debian/changelog
===================================================================
--- trunk/testset/manpages/debian/changelog     2008-01-03 04:30:43 UTC (rev 
1092)
+++ trunk/testset/manpages/debian/changelog     2008-01-03 04:39:27 UTC (rev 
1093)
@@ -4,6 +4,7 @@
   * debian/rules: 
     + Fix a little bug creating a file named 755 in the source dir.
     + Add a new binary with no english manpage.
+  * Sample changelog fixing spelling: publically -> publicly
 
  -- Marc 'HE' Brockschmidt <[EMAIL PROTECTED]>  Sat, 18 Dec 2004 13:32:29 +0100
 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to