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

lintian: r410 - in trunk: checks debian testset testset/maintainer-scripts/debian



Author: jeroen
Date: 2005-01-21 00:34:54 +0100 (Fri, 21 Jan 2005)
New Revision: 410

Modified:
   trunk/checks/changelog-file
   trunk/checks/changelog-file.desc
   trunk/debian/changelog
   trunk/testset/maintainer-scripts/debian/changelog
   trunk/testset/tags.foo++
   trunk/testset/tags.maintainer-scripts
Log:
* checks/changelog-file{,desc}:
  + [JvW] (new test) Bark on @toplevel email signatures in changelogs
  + [JvW] Display invalid debmake email address upon detection


Modified: trunk/checks/changelog-file
===================================================================
--- trunk/checks/changelog-file	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/checks/changelog-file	2005-01-20 23:34:54 UTC (rev 410)
@@ -237,8 +237,10 @@
     if (defined $prefix && defined $suffix && /^\Q$prefix\E\s*add-log-mailing-address:.*\Q$suffix\E$/) {
 	tag "debian-changelog-file-contains-obsolete-user-emacs-settings";
     }
-    if (/^\s*--[^<]*<[^>\@]+\@unknown>/) {
-       tag "debian-changelog-file-contains-debmake-default-email-address";
+    if (/^\s*--[^<]*<([^>\@]+\@unknown)>/) {
+       tag "debian-changelog-file-contains-debmake-default-email-address", $1;
+    } elsif (/^\s*--[^<]*<([^>\@]+\@[^>.]*)>/) {
+       tag "debian-changelog-file-contains-invalid-email-address", $1;
     }
 }
 close IN;

Modified: trunk/checks/changelog-file.desc
===================================================================
--- trunk/checks/changelog-file.desc	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/checks/changelog-file.desc	2005-01-20 23:34:54 UTC (rev 410)
@@ -91,6 +91,11 @@
 Info: The changelog file contains an email address (&lt;..@unknown&gt;)
  that was not updated to the maintainer's real address.
 
+Tag: debian-changelog-file-contains-invalid-email-address
+Type: error
+Info: The changelog file contains an invalid email address: the domain needs
+ at least one dot. This is looks like a mistake.
+
 Tag: debian-changelog-file-uses-obsolete-national-encoding
 Type: error
 Info: The Debian changelog file must be valid UTF-8, an encoding of

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/debian/changelog	2005-01-20 23:34:54 UTC (rev 410)
@@ -11,6 +11,9 @@
     + [JvW] Stop bugging us for exceptions, mmkay, just install overrides.
       Hints ..like this should be in the general documentation anyway, not in
       per-tag descriptions.
+  * checks/changelog-file{,desc}:
+    + [JvW] (new test) Bark on @toplevel email signatures in changelogs
+    + [JvW] Display invalid debmake email address upon detection
   * checks/common_data.pm:
     + [CW] Add tclx8.4 to %known_tclxs.
   * checks/description:

Modified: trunk/testset/maintainer-scripts/debian/changelog
===================================================================
--- trunk/testset/maintainer-scripts/debian/changelog	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/testset/maintainer-scripts/debian/changelog	2005-01-20 23:34:54 UTC (rev 410)
@@ -5,6 +5,12 @@
 
  -- Jeroen van Wolffelaar <jeroen@wolffelaar.nl>  Sun, 18 Apr 2004 02:05:29 +0200
 
+maintainer-scripts (6.1) unstable; urgency=low
+
+  * I made this entry on my local PC, oops ;)
+
+ -- Jeroen van Wolffelaar <jeroen@mordor>  Sat, 21 Feb 2004 18:29:37 +0100
+
 maintainer-scripts (6) unstable; urgency=low
 
   * added ldconfig calls to postrm to check test the postrm tests

Modified: trunk/testset/tags.foo++
===================================================================
--- trunk/testset/tags.foo++	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/testset/tags.foo++	2005-01-20 23:34:54 UTC (rev 410)
@@ -9,7 +9,7 @@
 E: foo++ source: uploader-name-missing Yama@gotchi
 E: foo++ source: wrong-debian-qa-address-set-as-maintainer Lintian Maintainer <debian-qa@lists.debian.org>
 E: foo++-helper: wrong-debian-qa-address-set-as-maintainer Lintian Maintainer <debian-qa@lists.debian.org>
-E: foo++: debian-changelog-file-contains-debmake-default-email-address
+E: foo++: debian-changelog-file-contains-debmake-default-email-address he@unknown
 E: foo++: debian-changelog-file-uses-obsolete-national-encoding at line 11
 E: foo++: no-copyright-file
 E: foo++: wrong-debian-qa-address-set-as-maintainer Lintian Maintainer <debian-qa@lists.debian.org>

Modified: trunk/testset/tags.maintainer-scripts
===================================================================
--- trunk/testset/tags.maintainer-scripts	2005-01-14 23:48:04 UTC (rev 409)
+++ trunk/testset/tags.maintainer-scripts	2005-01-20 23:34:54 UTC (rev 410)
@@ -2,6 +2,7 @@
 E: maintainer-scripts source: wrong-debian-qa-group-name QA group <packages@qa.debian.org>
 E: maintainer-scripts: control-file-has-bad-permissions config 0644 != 0755
 E: maintainer-scripts: debconf-config-not-executable
+E: maintainer-scripts: debian-changelog-file-contains-invalid-email-address jeroen@mordor
 E: maintainer-scripts: forbidden-config-interpreter #!/usr/bin/python
 E: maintainer-scripts: interpreter-without-predep control/config #!/usr/bin/python
 E: maintainer-scripts: maintainer-script-does-not-check-for-existence-of-wm-menu-config postinst:28



Reply to: