<p>On Feb 20, 2012 7:12 AM, "Stephen Kelly" <<a href="mailto:steveire@gmail.com">steveire@gmail.com</a>> wrote:<br>
><br>
> Dario Freddi wrote:<br>
><br>
> > 2012/2/19 Stephen Kelly <<a href="mailto:steveire@gmail.com">steveire@gmail.com</a>>:<br>
> >> Stephen Kelly wrote:<br>
> >><br>
> >>><br>
> >>> Hi there,<br>
> >>><br>
> >>> I was reviewing the changes in the frameworks branch from yesterday.<br>
> >>> Something I noticed was that there are a lot of merge commits that don't<br>
> >>> need to exist.<br>
> >><br>
> >> Ugh. Yet more of this just appeared... Recent history in the frameworks<br>
> >> branch looks far more complex than it is and is harder to follow.<br>
> >><br>
> >> There are too many people unaware that they're doing it maybe...<br>
> ><br>
> > What about having a small volunteer day in which we draft some<br>
> > policies for git commits and eventually implement some hooks to avoid<br>
> > these (and other) things happening?<br>
> ><br>
><br>
> There's some ideas for such hooks on the Internet already:<br>
><br>
> <a href="http://stackoverflow.com/questions/2039773/have-remote-git-repository-">http://stackoverflow.com/questions/2039773/have-remote-git-repository-</a><br>
> refuse-merge-commits-on-push<br>
><br>
> I don't know ruby or shell scripting well enough to write something sysadmin<br>
> would accept, but I would definitely welcome such a hook.</p>
<p>First, our hooks are in python - so any changes would need to be in Python as well.</p>
<p>Second, there is a legitimate use for pushing merge commits - namely integrating a seperate remote branch. I have no idea if it would be possible to determine if this is a local only branch merge or not. </p>
<p>Third, please ensure that you disable the previously mentioned auto-rebase on git pull if you are merging a reemote branch, otherwise you will duplicate many commits, in addition to triggering the hooks for all those commits (which will not happen if you just merge). The hooks could even decide to reject your push, as it will contain too many new commits.</p>

<p>><br>
> Thanks,<br>
><br>
> Steve.<br>
><br>
></p>
<p>Regards,<br>
Ben Cooksley<br>
KDE Sysadmin.</p>