mirror of
https://git.launchpad.net/ubuntu-dev-tools
synced 2025-05-11 08:51:29 +00:00
sponsor-patch: Use syncpackage instead of subscribing ubuntu-archive for sync requests, because syncpackage supports sponsorship now.
This commit is contained in:
parent
9482e7b9f9
commit
58e6dfbbdb
7
debian/changelog
vendored
7
debian/changelog
vendored
@ -1,3 +1,10 @@
|
|||||||
|
ubuntu-dev-tools (0.138) UNRELEASED; urgency=low
|
||||||
|
|
||||||
|
* sponsor-patch: Use syncpackage instead of subscribing ubuntu-archive for
|
||||||
|
sync requests, because syncpackage supports sponsorship now.
|
||||||
|
|
||||||
|
-- Benjamin Drung <bdrung@debian.org> Sat, 10 Dec 2011 12:46:33 +0100
|
||||||
|
|
||||||
ubuntu-dev-tools (0.137) unstable; urgency=low
|
ubuntu-dev-tools (0.137) unstable; urgency=low
|
||||||
|
|
||||||
[ Stefano Rivera ]
|
[ Stefano Rivera ]
|
||||||
|
@ -14,9 +14,6 @@ builds it, runs
|
|||||||
.BR lintian (1)
|
.BR lintian (1)
|
||||||
and, after review and confirmation, can upload it.
|
and, after review and confirmation, can upload it.
|
||||||
|
|
||||||
For syncs, sponsor\-patch will follow the usual procedure of commenting
|
|
||||||
on \fIbug\fR, marking it confirmed, and subscribing \fBubuntu-archive\fR.
|
|
||||||
|
|
||||||
\fBsponsor\-patch\fR can be used for sponsoring patches, syncs and
|
\fBsponsor\-patch\fR can be used for sponsoring patches, syncs and
|
||||||
merges from Debian, SRUs, and creating debdiffs from patches.
|
merges from Debian, SRUs, and creating debdiffs from patches.
|
||||||
If \fIbug\fR has multiple patches or branches linked, it will prompt the
|
If \fIbug\fR has multiple patches or branches linked, it will prompt the
|
||||||
|
@ -437,14 +437,12 @@ class SourcePackage(object):
|
|||||||
|
|
||||||
return lintian_filename
|
return lintian_filename
|
||||||
|
|
||||||
def sync(self, upload, bug_number, keyid):
|
def sync(self, upload, bug_number, requester):
|
||||||
"""Does a sync of the source package."""
|
"""Does a sync of the source package."""
|
||||||
|
|
||||||
if upload == "ubuntu":
|
if upload == "ubuntu":
|
||||||
cmd = ["syncpackage", self._package, "-b", str(bug_number),
|
cmd = ["syncpackage", self._package, "-b", str(bug_number), "-f",
|
||||||
"-V", str(self._version)]
|
"-s", requester, "-V", str(self._version)]
|
||||||
if keyid is not None:
|
|
||||||
cmd += ["-k", keyid]
|
|
||||||
Logger.command(cmd)
|
Logger.command(cmd)
|
||||||
if subprocess.call(cmd) != 0:
|
if subprocess.call(cmd) != 0:
|
||||||
Logger.error("Syncing of %s %s failed.", self._package,
|
Logger.error("Syncing of %s %s failed.", self._package,
|
||||||
@ -452,7 +450,7 @@ class SourcePackage(object):
|
|||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
else:
|
else:
|
||||||
# FIXME: Support this use case!
|
# FIXME: Support this use case!
|
||||||
Logger.error("Uploading a synced package other than to ubuntu "
|
Logger.error("Uploading a synced package other than to Ubuntu "
|
||||||
"is not supported yet!")
|
"is not supported yet!")
|
||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
return True
|
return True
|
||||||
|
@ -277,8 +277,7 @@ def sponsor_patch(bug_number, build, builder, edit, keyid, lpinstance, update,
|
|||||||
previous_version)
|
previous_version)
|
||||||
|
|
||||||
if successful:
|
if successful:
|
||||||
#if source_package.sync(upload, bug_number, keyid):
|
if source_package.sync(upload, bug_number, bug.owner.name):
|
||||||
if source_package.ack_sync(upload, task.get_lp_task(), launchpad):
|
|
||||||
return
|
return
|
||||||
else:
|
else:
|
||||||
edit = True
|
edit = True
|
||||||
|
Loading…
x
Reference in New Issue
Block a user