On Sat, Sep 05, 2009 at 02:28:46AM -0400, Jeff King wrote:
I see. I still think you may want to improve "commit --dry-run" with a
plumbing format, though, instead of "git status". Then it would
automagically support "--amend", as well as other dry-run things (e.g.,
"git commit --dry-run --porcelain --amend foo.c"). And not having looked
at the code, I would guess it is a one-liner patch to switch the "output
format" flag that commit passes to the wt-status.c code.
OK, it was a bit more complex than that. But here is a series which does
a few things. It is still missing a few bits, so is RFC.
These first two are unrelated fixups that I noticed while working.
[1/6]: status: typo fix in usage
[2/6]: docs: note that status configuration affects only long format
These are the --porcelain patches we discussed. The first two are
obviously cleanup.
[3/6]: status: refactor short-mode printing to its own function
[4/6]: status: refactor format option parsing
[5/6]: status: add --porcelain output format
This brings the new formats to "commit --dry-run" to handle your case.
Conceptually, it could come before (or instead) of 4/6 and 5/6, but as
it adds both --short and --porcelain, there is an obvious dependency.
[6/6]: commit: support alternate status formats
-Peff