From 186375e8b2213f2f03921f3fd689ef0db8cd3799 Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Wed, 25 Mar 2015 17:36:14 +0000 Subject: [PATCH] Obviously, this should come first. --- roles/git/hooks/files/post-receive-fedmsg | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/roles/git/hooks/files/post-receive-fedmsg b/roles/git/hooks/files/post-receive-fedmsg index 4169b91b53..5ad22d1e04 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),