[PATCH] post-receive: exclude commits from the patch update step
Brian Norris
computersforpeace at gmail.com
Fri Jul 4 11:20:14 EST 2014
On second (third?) thought...
On Thu, Jul 3, 2014 at 5:56 PM, Brian Norris
<computersforpeace at gmail.com> wrote:
> On Thu, Jul 3, 2014 at 5:49 PM, Jeremy Kerr <jk at ozlabs.org> wrote:
> ...
>>> @@ -39,7 +45,8 @@ set_patch_state()
>>> update_patches()
>>> {
>>> local cnt; cnt=0
>>> - for rev in $(git rev-list --no-merges --reverse ${1}..${2}); do
>>> + for rev in $(git rev-parse --not ${EXCLUDE} |
>>> + git rev-list --stdin --no-merges --reverse ${1}..${2}); do
>
> I think I can probably combine the rev-parse and rev-list to avoid the
> extra pipe:
>
> git rev-list --no-merges --reverse ${1}..${2} --not ${EXCLUDE}
>
> I'll test and resend, if that looks OK.
Actually, I think the rev-parse serves as a little extra parameter
checking. Unless someone can pin a good reason one way or the other,
I'll stick with my original patch as I sent it. And I've been testing
that version for a few weeks now.
Brian
More information about the Patchwork
mailing list