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

[lintian] 01/01: Stop emitting {maintainer, uploader}-address-causes-mail-loops for @packages.debian.org addresses. (Closes: #871575)



This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 56864ad575f70ad83a41229b06175918a2d81da4
Author: Chris Lamb <lamby@debian.org>
Date:   Thu Aug 10 09:11:34 2017 -0400

    Stop emitting {maintainer,uploader}-address-causes-mail-loops for @packages.debian.org addresses. (Closes: #871575)
---
 checks/fields.desc                     | 4 +---
 debian/changelog                       | 4 ++++
 lib/Lintian/Check.pm                   | 7 ++-----
 t/tests/fields-maintainer-bounces/tags | 2 --
 4 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/checks/fields.desc b/checks/fields.desc
index d5b87e6..bfab0bd 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -196,9 +196,7 @@ Tag: maintainer-address-causes-mail-loops-or-bounces
 Severity: serious
 Certainty: certain
 Info: The maintainer e-mail address either loops back to itself because
- it is set to package@packages.debian.org or
- package@packages.qa.debian.org  or references an email address
- (typically a mailing list) which is  known to bounce mails. Policy
+ it is set to an address which is known to bounce mails.  Policy
  states: The email address given in the Maintainer control field must
  accept  mail from those role accounts in Debian used to send automated
  mails  regarding the package. This includes non-spam mail from the bug-
diff --git a/debian/changelog b/debian/changelog
index 2e843de..28e972b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -75,6 +75,10 @@ lintian (2.5.53) UNRELEASED; urgency=medium
     + [CL] Apply patch from Maia Everett to avoid British spelling when
       using en_US locale.  (Closes: #868897)
 
+  * lib/Lintian/Check.pm:
+    + [CL] Stop emitting {maintainer,uploader}-address-causes-mail-loops
+      for @packages.debian.org addresses.  (Closes: #871575)
+
   * helpers/coll/objdump-info-helper:
     + [CL] Apply patch from Steve Langasek to accomodate binutils 2.29
       outputting symbols in a different format on ppc64el.
diff --git a/lib/Lintian/Check.pm b/lib/Lintian/Check.pm
index cb2a380..8fd0c53 100644
--- a/lib/Lintian/Check.pm
+++ b/lib/Lintian/Check.pm
@@ -191,11 +191,8 @@ sub check_maintainer {
             tag "$field-address-is-root-user", $maintainer;
         }
 
-        if (
-            ($field ne 'changed-by')
-            and (  $mail =~ /\@packages\.(?:qa\.)?debian\.org/i
-                or $KNOWN_BOUNCE_ADDRESSES->known($mail))
-          ) {
+        if (($field ne 'changed-by') and $KNOWN_BOUNCE_ADDRESSES->known($mail))
+        {
             tag "$field-address-causes-mail-loops-or-bounces", $maintainer;
         }
 
diff --git a/t/tests/fields-maintainer-bounces/tags b/t/tests/fields-maintainer-bounces/tags
index e1b1e68..f5906d7 100644
--- a/t/tests/fields-maintainer-bounces/tags
+++ b/t/tests/fields-maintainer-bounces/tags
@@ -1,4 +1,2 @@
 E: fields-maintainer-bounces source: maintainer-address-causes-mail-loops-or-bounces Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
-E: fields-maintainer-bounces source: uploader-address-causes-mail-loops-or-bounces I Will Loop <fields-maintainer-bounces@packages.debian.org>
-E: fields-maintainer-bounces source: uploader-address-causes-mail-loops-or-bounces I Will Loop Too <fields-maintainer-bounces@PACKAGES.QA.DEBIAN.ORG>
 E: fields-maintainer-bounces: maintainer-address-causes-mail-loops-or-bounces Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: