launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #29497
[Merge] ~cjwatson/launchpad-buildd:sourcepackagerecipe-remove-distroseries-name into launchpad-buildd:master
Colin Watson has proposed merging ~cjwatson/launchpad-buildd:sourcepackagerecipe-remove-distroseries-name into launchpad-buildd:master.
Commit message:
sourcepackagerecipe: Remove the old "distroseries_name" argument
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~cjwatson/launchpad-buildd/+git/launchpad-buildd/+merge/435135
This duplicated the common "series" argument.
--
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad-buildd:sourcepackagerecipe-remove-distroseries-name into launchpad-buildd:master.
diff --git a/debian/changelog b/debian/changelog
index b306ecb..7e6ed12 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+launchpad-buildd (226) UNRELEASED; urgency=medium
+
+ * Remove the old "distroseries_name" argument from the sourcepackagerecipe
+ manager, which duplicated the common "series" argument.
+
+ -- Colin Watson <cjwatson@xxxxxxxxxx> Wed, 04 Jan 2023 13:33:59 +0000
+
launchpad-buildd (225) focal; urgency=medium
* Fix add-trusted-keys regression due to Backend.open calling "lxc exec"
diff --git a/lpbuildd/sourcepackagerecipe.py b/lpbuildd/sourcepackagerecipe.py
index 15d5d65..9b5520f 100644
--- a/lpbuildd/sourcepackagerecipe.py
+++ b/lpbuildd/sourcepackagerecipe.py
@@ -77,7 +77,6 @@ class SourcePackageRecipeBuildManager(DebianBuildManager):
self.author_name = extra_args['author_name']
self.author_email = extra_args['author_email']
self.archive_purpose = extra_args['archive_purpose']
- self.distroseries_name = extra_args['distroseries_name']
self.git = extra_args.get('git', False)
super().initiate(files, chroot, extra_args)
@@ -92,7 +91,7 @@ class SourcePackageRecipeBuildManager(DebianBuildManager):
args.append("--git")
args.extend([
self._buildid, self.author_name.encode('utf-8'),
- self.author_email, self.suite, self.distroseries_name,
+ self.author_email, self.suite, self.series,
self.component, self.archive_purpose])
self.runSubProcess(self.build_recipe_path, args)
diff --git a/lpbuildd/tests/test_sourcepackagerecipe.py b/lpbuildd/tests/test_sourcepackagerecipe.py
index 2cf7fd3..52493f8 100644
--- a/lpbuildd/tests/test_sourcepackagerecipe.py
+++ b/lpbuildd/tests/test_sourcepackagerecipe.py
@@ -72,7 +72,6 @@ class TestSourcePackageRecipeBuildManagerIteration(TestCase):
'author_name': 'Steve\u1234',
'author_email': 'stevea@xxxxxxxxxxx',
'archive_purpose': 'puppies',
- 'distroseries_name': 'maverick',
'archives': [
'deb http://archive.ubuntu.com/ubuntu maverick main universe',
'deb http://ppa.launchpad.net/launchpad/bzr-builder-dev/'