launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad-buildd:sbuild-remove-mailto into launchpad-buildd:master.

Commit message:
Stop setting $mailto in .sbuildrc

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad-buildd/+git/launchpad-buildd/+merge/392876

This works around LP #1859010.  Setting $mailto used to be required, but hasn't been since sbuild 0.65.1 or so, before xenial.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad-buildd:sbuild-remove-mailto into launchpad-buildd:master.

diff --git a/debian/changelog b/debian/changelog
index 2d844f7..d9471b6 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+launchpad-buildd (194) UNRELEASED; urgency=medium
+
+  * Stop setting $mailto in .sbuildrc, to work around LP #1859010.
+
+ -- Colin Watson <cjwatson@xxxxxxxxxx>  Tue, 27 Oct 2020 13:22:05 +0000
+
 launchpad-buildd (193) bionic; urgency=medium
 
   * Fix handling of bytes arguments passed to BuildManager.runSubProcess.
diff --git a/sbuildrc b/sbuildrc
index c49ada9..3f45546 100644
--- a/sbuildrc
+++ b/sbuildrc
@@ -1,6 +1,3 @@
-# Mail address where logs are sent to (mandatory, no default!)
-$mailto = "buildd-maint\@canonical.com";
-
 # Maintainer name to use in .changes files (mandatory, no default!)
 my $fqdn=`hostname --fqdn`;
 chomp $fqdn;