Author: adam-guest
Date: 2008-09-08 20:36:06 +0000 (Mon, 08 Sep 2008)
New Revision: 1617

Modified:
   trunk/debian/changelog
   trunk/scripts/bts.pl
Log:
No longer automatically CC the generated mail to the bug address if a
comment was made. Similar functionality may be implemented in the BTS
at a later date but in the meantime we'll avoid creating noise in the
bug logs. (Closes: #497748)

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-09-08 17:46:37 UTC (rev 1616)
+++ trunk/debian/changelog      2008-09-08 20:36:06 UTC (rev 1617)
@@ -17,6 +17,10 @@
     + Mention in the manpage that BTS_ONLY_NEW corresponds to --only-new.
     + Ensure that the "clone" command is always passed at least one new bug ID
       (Closes: #497826) and that the new IDs are negative numbers.
+    + No longer automatically CC the generated mail to the bug address if a
+      comment was made. Similar functionality may be implemented in the BTS
+      at a later date but in the meantime we'll avoid creating noise in the
+      bug logs. (Closes: #497748)
   * checkbashisms: Flag "read without variable" and "read with options other
     than -r" separately to ensure that all uses are correctly matched. Thanks
     to Frank Lichtenheld for pointing this out via a related bug in lintian.

Modified: trunk/scripts/bts.pl
===================================================================
--- trunk/scripts/bts.pl        2008-09-08 17:46:37 UTC (rev 1616)
+++ trunk/scripts/bts.pl        2008-09-08 20:36:06 UTC (rev 1617)
@@ -195,9 +195,6 @@
 program, unless the comment is quoted.  (Something like "bts
 severity #85942 normal" will not be treated as a comment!)
 
-In most cases, adding a comment will cause the generated mail to be CCed
-to the bug report, in addition to [EMAIL PROTECTED]
-
 You can specify multiple commands by separating them with a single dot,
 rather like B<update-rc.d>; a single comma may also be used; all the
 commands will then be sent in a single mail. For example (quoting where
@@ -2192,11 +2189,6 @@
     $charset =~ s/^ANSI_X3\.4-19(68|86)$/US-ASCII/;
     $subject = MIME_encode_mimewords($subject, 'Charset' => $charset);
 
-    # If there were comments, we CC each of the bugs
-    if (keys %ccbugs && length(join('', @comment))) {
-       $ccemail .= ", " if length $ccemail;
-       $ccemail .= join("[EMAIL PROTECTED], ", sort (keys %ccbugs)) . "[EMAIL 
PROTECTED]";
-    }
     if (keys %ccpackages && $packagesserver) {
        $ccemail .= ", " if length $ccemail;
        $ccemail .= join("[EMAIL PROTECTED], ", sort (keys %ccpackages))



-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to