> 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
_______________________________________________
infrastructure mailing list -- [email protected]
To unsubscribe send an email to [email protected]
