Re: [PATCHv2] git-status: show short sequencer state
- From
Phil Hord <phil.hord@gmail.com>
- Date
- Oct 29, 2012, 22:26 UTC
- Message-ID
- <CABURp0qBRvZQvnBbOraQ7c7DRg8v0TjnY+MOGYaWnWwjCqi23Q@mail.gmail.com>
- In-Reply-To
- <20121029214103.GD20513@sigill.intra.peff.net>
On Mon, Oct 29, 2012 at 5:41 PM, Jeff King <peff@peff.net> wrote:
Show 40 quoted lines
> On Mon, Oct 29, 2012 at 02:05:14PM -0400, Phil Hord wrote: > >> I'm currently splitting this out into a series and reconsidering some >> of it along the way. I need some guidance. >> >> I want to support these two modes: >> >> A. 'git status --short' with sequence tokens added: >> ## conflicted >> ## merge >> ?? untracked-workdir-file >> etc. >> >> B. Same as (A) but without workdir status: >> ## conflicted >> ## merge >> >> The user who wants 'A' would initiate it like this: >> git status --sequencer >> or >> git status -S >> >> How do I spell the options for 'B'? I have come up with these three >> possibilities: >> git --sequencer-only # Another switch >> git --sequencer=only # An OPTARG parser >> git -S -S # like git-diff -C -C, an OPT_COUNTUP > > Might it be easier to spell 'A' as: > > git status --short -S > > and B as: > > git status -S > > this is sort of like how "-b" works (except you cannot currently ask for > it separately, but arguably you could). If we have a proliferation of > such options, then we might need config to help turn them on all the > time (I'd guess people are probably already using aliases to do this).
I think I like this path.
I expect a common idiom to be 'git status -S --porcelain --null', and both --porcelain and --null imply --short. I think I can still do The Right Thing, but the code is starting to spaghettify. I'll take a crack at it.
Thanks.
P