diff options
| author | Ralph Bean <rbean@redhat.com> | 2015-03-25 17:36:14 +0000 |
|---|---|---|
| committer | Ralph Bean <rbean@redhat.com> | 2015-03-25 17:36:26 +0000 |
| commit | 186375e8b2213f2f03921f3fd689ef0db8cd3799 (patch) | |
| tree | 54e009c159adce57e66e46afde45e2ecc73a14ed | |
| parent | 101eb8d230503e81c097b2e313e0a2dc2bcc51cd (diff) | |
| download | ansible-186375e8b2213f2f03921f3fd689ef0db8cd3799.tar.gz ansible-186375e8b2213f2f03921f3fd689ef0db8cd3799.tar.xz ansible-186375e8b2213f2f03921f3fd689ef0db8cd3799.zip | |
Obviously, this should come first.
| -rwxr-xr-x | roles/git/hooks/files/post-receive-fedmsg | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/roles/git/hooks/files/post-receive-fedmsg b/roles/git/hooks/files/post-receive-fedmsg index 4169b91b5..5ad22d1e0 100755 --- a/roles/git/hooks/files/post-receive-fedmsg +++ b/roles/git/hooks/files/post-receive-fedmsg @@ -119,6 +119,9 @@ for line in lines: print "* Publishing information for %i commits" % len(commits) for commit in commits: + if commit is None: + continue + # Keep track of whether or not we have already published this commit on # another branch or not. It is conceivable that someone could make a # commit to a number of branches, and push them all at the same time. @@ -130,9 +133,6 @@ for line in lines: commit['seen'] = False seen.append(commit['rev']) - if commit is None: - continue - fedmsg.publish( topic="receive", msg=dict(commit=commit), |