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 epelif not refname.startswith(('refs/heads/master','refs/heads/f')):
continuenew_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