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

[SCM] Debian package checker branch, master, updated. 2.2.18-97-g3cfffb8

The following commit has been merged in the master branch:
commit e7eb6c44d45e2ab7c7a7fde9f4af10f147c641e1
Author: Russ Allbery <rra@debian.org>
Date:   Sat Dec 26 17:04:18 2009 -0800

    Split capitalization checks in description to a separate tag
    
    * checks/description{,.desc}:
      + [RA] Split capitalization-error-in-description to a separate tag and
        downgrade its certainty to wild-guess.

diff --git a/checks/description b/checks/description
index 90c7d2b..0b676c7 100644
--- a/checks/description
+++ b/checks/description
@@ -186,7 +186,7 @@ unless ($info->field('homepage') or $flagged_homepage) {
 
 if ($description) {
     spelling_check('spelling-error-in-description', $description);
-    spelling_check_picky('spelling-error-in-description', $description);
+    spelling_check_picky('capitalization-error-in-description', $description);
 }
 
 }
diff --git a/checks/description.desc b/checks/description.desc
index a2ed61e..22ca72a 100644
--- a/checks/description.desc
+++ b/checks/description.desc
@@ -136,11 +136,24 @@ Info: The extended description contains a "Homepage:" pseudo-header
 Tag: spelling-error-in-description
 Severity: minor
 Certainty: certain
-Info: Lintian found a spelling or capitalization error in the package
- description.  Lintian has a list of common misspellings that it looks
- for.  It does not have a dictionary like a spelling checker does.  It is
- particularly picky about spelling and capitalization in package
- descriptions since they're very visible to end users.
+Info: Lintian found a spelling error in the package description.  Lintian
+ has a list of common misspellings that it looks for.  It does not have a
+ dictionary like a spelling checker does.  It is particularly picky about
+ spelling and capitalization in package descriptions since they're very
+ visible to end users.
+
+Tag: capitalization-error-in-description
+Severity: minor
+Certainty: wild-guess
+Info: Lintian found a possible capitalization error in the package
+ description.  Lintian has a list of common capitalization errors,
+ primarily of upstream projects, that it looks for.  It does not have a
+ dictionary like a spelling checker does.
+ .
+ This is a particularly picky check of capitalization in package
+ descriptions, since they're very visible to end users, but it will have
+ false positives for project names used in a context where they should be
+ lowercase, such as package names or executable names.
 
 Tag: description-contains-duplicated-word
 Severity: normal
diff --git a/debian/changelog b/debian/changelog
index 277bea4..9573f3b 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,8 @@ lintian (2.3.0) UNRELEASED; urgency=low
 
   * Summary of tag changes:
     + Added:
+      - capitalization-error-in-description
+        (split from spelling-error-in-description)
       - debian-rules-automatically-updates-control
       - duplicate-in-relation-field (pedantic)
       - duplicated-compressed-file
@@ -60,8 +62,10 @@ lintian (2.3.0) UNRELEASED; urgency=low
       Thanks, Ahmed El-Mahmoudy.  (Closes: #556022)
   * checks/debian-readme.desc:
     + [RA] Downgrade spelling tag to severity: minor.
-  * checks/description.desc:
+  * checks/description{,.desc}:
     + [RA] Downgrade spelling tag to severity: minor.
+    + [RA] Split capitalization-error-in-description to a separate tag and
+      downgrade its certainty to wild-guess.
   * checks/fields{,.desc}:
     + [ADB] Re-order the fields of build-depends-on-build-essential so that
       all of the descriptive text is included.  Thanks, Jonathan Wiltshire.
diff --git a/t/COVERAGE b/t/COVERAGE
index 7e59289..fc72dbd 100644
--- a/t/COVERAGE
+++ b/t/COVERAGE
@@ -1,4 +1,4 @@
-Last generated 2009-12-26
+Last generated 2009-12-27
 
 The following tags are not tested by the test suite:
 
diff --git a/t/tests/description-general/debian/debian/control.in b/t/tests/description-general/debian/debian/control.in
index 51870a0..0946989 100644
--- a/t/tests/description-general/debian/debian/control.in
+++ b/t/tests/description-general/debian/debian/control.in
@@ -50,7 +50,7 @@ Depends: $\{shlibs:Depends\}, $\{misc:Depends\}
 Description: some Lintian test package
  some Lintian test package
  .
- Some mroe stuff about this test package.
+ Some mroe stuff about this debian test package.
  .
  Homepage: <http://lintian.debian.org/>
 
diff --git a/t/tests/description-general/desc b/t/tests/description-general/desc
index b931f02..153d0d0 100644
--- a/t/tests/description-general/desc
+++ b/t/tests/description-general/desc
@@ -3,6 +3,7 @@ Sequence: 6000
 Version: 1.0
 Description: Tests of various description tags
 Test-For:
+ capitalization-error-in-description
  description-contains-dh-make-perl-template
  description-contains-duplicated-word
  description-contains-homepage
diff --git a/t/tests/description-general/tags b/t/tests/description-general/tags
index ac8f17a..0e80345 100644
--- a/t/tests/description-general/tags
+++ b/t/tests/description-general/tags
@@ -8,6 +8,7 @@ E: description-general: description-contains-tabs
 E: description-general: description-is-debmake-template
 E: description-general: description-starts-with-package-name
 I: description-general-3: using-first-person-in-description line 5: I
+I: description-general-4: capitalization-error-in-description debian Debian
 I: description-general: description-synopsis-might-not-be-phrased-properly
 W: description-general-2: description-starts-with-leading-spaces
 W: description-general-2: possible-unindented-list-in-extended-description

-- 
Debian package checker
Reply to: