[devscripts] 01/01: bts: support "newcomer" tag w/ backward compatibility for "gift"

[Stefano Zacchiroli]

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

zack pushed a commit to branch master
in repository devscripts.

commit 5c7978d4508fcb895463b2e9ce828e4dd8f79be4
Author: Stefano Zacchiroli <zack at debian.org>
Date:   Sat Nov 15 10:44:57 2014 +0100

    bts: support "newcomer" tag w/ backward compatibility for "gift"
    
    - add support for the "newcomer" tag
    
    - backward compatibility for the old "gift" usertag: drop documentation for it
      (so that we do not encourage further usage of it), but do both "gift" and
      "newcomer" (user)tagging for the time being when invoked with +gift
---
 debian/changelog |  7 +++++++
 scripts/bts.pl   | 13 ++++++-------
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 87f4b73..1d1d0a6 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -27,6 +27,13 @@ devscripts (2.14.11) UNRELEASED; urgency=medium
   * bts: Support the "stretch", "buster", "stretch-ignore" and "buster-ignore"
     tags.
 
+  [ Stefano Zacchiroli ]
+  * bts:
+    + Support for the "newcomer" tag
+    + Backward compatibility for the old "gift" usertag: drop
+      documentation for it, but do both gift and newcomer (user)tagging for
+      the time being
+
  -- Paul Wise <pabs at debian.org>  Sun, 19 Oct 2014 17:27:24 +0800
 
 devscripts (2.14.10) unstable; urgency=medium
diff --git a/scripts/bts.pl b/scripts/bts.pl
index afef72c..0d60fef 100755
--- a/scripts/bts.pl
+++ b/scripts/bts.pl
@@ -156,7 +156,7 @@ our (@gTags, @valid_tags, %valid_tags);
            "potato", "woody", "sid", "help", "security", "upstream",
            "pending", "sarge", "sarge-ignore", "experimental", "d-i",
            "confirmed", "ipv6", "lfs", "fixed-in-experimental",
-           "fixed-upstream", "l10n", "etch", "etch-ignore",
+           "fixed-upstream", "l10n", "newcomer", "etch", "etch-ignore",
            "lenny", "lenny-ignore", "squeeze", "squeeze-ignore",
            "wheezy", "wheezy-ignore", "jessie", "jessie-ignore",
            "stretch", "stretch-ignore", "buster", "buster-ignore",
@@ -1542,12 +1542,6 @@ must be specified, unless the B<=> flag is used, where the command
 
 will remove all tags from the specified I<bug>.
 
-As a special case, the unofficial B<gift> tag name is supported in
-addition to official tag names. B<gift> is used as a shorthand for the
-B<gift> usertag; see L<https://wiki.debian.org/qa.debian.org/GiftTag>.
-Adding/removing the B<gift> tag will add/remove the B<gift> usertag,
-belonging to the "debian-qa at lists.debian.org" user.
-
 Adding/removing the B<security> tag will add "team\@security.debian.org"
 to the Cc list of the control email.
 
@@ -1584,6 +1578,11 @@ sub bts_tags {
 	    if ($gift_flag eq '=') {
 		$gift_flag = '+';
 	    }
+	    # Backward compatibility: do both gift usertagging and newcomer
+	    # tagging. Gifting should be removed after a suitable migration
+	    # time.  See https://wiki.debian.org/qa.debian.org/GiftTag header
+	    # for more info.
+	    mailbts("tagging $bug", "tags $bug + newcomer");
 	    mailbts("gifting $bug",
 		"user debian-qa\@lists.debian.org\nusertag $bug $gift_flag gift");
 	    next;

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