"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

From:
Stefan Sperling <stsp@stsp.name>
Subject:
Re: fix staging of multiple files with -p
To:
Sebastien Marie <semarie@online.fr>
Cc:
gameoftrees@openbsd.org
Date:
Mon, 28 Oct 2019 11:32:57 +0100

Download raw body.

Thread
On Sun, Oct 27, 2019 at 02:21:22PM +0100, Sebastien Marie wrote:
> On Thu, Oct 24, 2019 at 06:18:19PM +0200, Stefan Sperling wrote:
> > There's a bug in stage -p where it fails if multiple files are involved.
> > 
> [...]
> >
> > Based on the above:
> > 
> > 1) If a file has no changes to stage in the -p case, move on to the
> > next file instead of reporting an error.
> > 
> > 2) If there is nothing to stage (with or without -p), print an error
> > and use an exit code != 0. The related test failed to take the exit
> > code into account, fix this.
> > 
> > ok?
> 
> the rational is good. the diff seems ok too.
> 
> ok semarie@

Committed. Thanks!