On Thu, Oct 20, 2016 at 12:40:29AM -0000, Josh Boyer wrote:
I'll rename and adjust the file if/when we'll all be ok with it (I'm fine
with
re-sending it for review then).
Sure, this should do it:
diff --git a/ roles/git/hooks/files/post-receive-secondaryarch b/
roles/git/hooks/files/post-receive-secondaryarch
index 9ad165f..189c69c 100755
--- a/ roles/git/hooks/files/post-receive-secondaryarch
+++ b/ roles/git/hooks/files/post-receive-secondaryarch
@@ -147,6 +147,11 @@ def run_as_post_receive_hook():
print('Received:', line.strip())
(oldrev, newrev, refname) = line.strip().split(' ', 2)
+ # Skip all the branches that are not master and do not start with an
'f'
+ # ie: personal branches and epel
+ if not refname.startswith(('refs/heads/master',
'refs/heads/f')):
+ continue
+
new_commits_list = get_revs_between(oldrev, newrev, abspath, refname)
if DEBUG:
print('List of commits:', new_commits_list)
Sounds like a good list for me, are the people interested in these emails
subscribed there? (I suspect they are, but better safe than sorry :))
The list is extremely low traffic. In the absence of any kind of current postings,
it's hard to really tell for sure. I think using the list is fine and if for some
reason we get a number of complaints we can adjust. I don't expect that to happen
though.
josh