Re: [PATCH 2/2] revert: don't print "Finished one cherry-pick." if commit failed

From: Christian Couder <chriscool@tuxfamily.org>
Date: Jul 13, 2010, 21:06 UTC
Message-ID: <201007132306.33828.chriscool@tuxfamily.org>
In-Reply-To: <20100712163934.GA1869@burratino>

On Monday 12 July 2010 18:39:35 Jonathan Nieder wrote:

Show 5 quoted lines

> Christian Couder wrote:
> > And, while at it, refactor commit code into a new run_git_commit()
> > function.
> 
> Probably should be two patches.

Ok, I will split it in two patches.

Show 18 quoted lines

> > +++ b/builtin/revert.c
> > @@ -492,48 +518,17 @@ static int do_pick_commit(void)
> > 
> >  			mebuf.buf, help_msg());
> >  		
> >  		rerere(allow_rerere_auto);
> >  	
> >  	} else {
> > 
> > -		fprintf(stderr, "Finished one %s.\n", mebuf.buf);
> > +		if (!no_commit)
> > +			res = run_git_commit(defmsg);
> > +		if (!res)
> > +			fprintf(stderr, "Finished one %s.\n", mebuf.buf);
> > 
> >  	}
> 
> Ack for this part.  Thanks.

Thanks, Christian.