Freeze break request: pkgdb_sync should only remove master from todo if it was requested
Pierre-Yves Chibon
pingou at pingoured.fr
Wed Feb 25 12:48:12 UTC 2015
On Wed, Feb 25, 2015 at 06:36:27AM -0500, Patrick Uiterwijk wrote:
> Hi,
>
> So another freeze break request, +1s appreciated.
> This change is the only thing needed to make non-master-repos an option.
> Currently, the code always removes master from the requested branches, and crashes if it wasn't there.
> I'm just adding a check to only remove it (and announce the package) if it was requested.
>
>
> commit d230b5701575879e757f2c4aeb8eed62c319d9e6
> Author: Patrick Uiterwijk <puiterwijk at redhat.com>
> Date: Wed Feb 25 11:34:07 2015 +0000
>
> Only remove master from request if it was requested
>
> Signed-off-by: Patrick Uiterwijk <puiterwijk at redhat.com>
>
> diff --git a/roles/distgit/templates/pkgdb_sync_git_branches.py b/roles/distgit/templates/pkgdb_sync_git_branches.py
> index c931d26..ced139b 100644
> --- a/roles/distgit/templates/pkgdb_sync_git_branches.py
> +++ b/roles/distgit/templates/pkgdb_sync_git_branches.py
> @@ -189,16 +189,17 @@ def branch_package(pkgname, branches):
> if not os.path.exists(
> os.path.join(GIT_FOLDER, '%s.git' % pkgname)):
> _invoke(SETUP_PACKAGE, [pkgname])
> - branches.remove('master') # SETUP_PACKAGE creates master
> - fedmsg.publish(
> - topic='branch',
> - modname='git',
> - msg=dict(
> - agent='pkgdb',
> - name=pkgname,
> - branch='master',
> - ),
> - )
> + if 'master' in branches:
> + branches.remove('master') # SETUP_PACKAGE creates master
> + fedmsg.publish(
> + topic='branch',
> + modname='git',
> + msg=dict(
> + agent='pkgdb',
> + name=pkgname,
> + branch='master',
> + ),
> + )
Could you expand a little about this fedmsg message?
We don't create the branch (created by SETUP_PACKAGE) but we still send a
message that we did?
Pierre
More information about the infrastructure
mailing list