Re: [PATCH v6 0/9] status: V2 porcelain status

2016-08-11 Thread Junio C Hamano
Jeff Hostetler  writes:

> From: Jeff Hostetler 
>
> This patch series adds porcelain V2 format to status.
> This provides detailed information about file changes
> and about the current branch.

Everything up to 8/9 I didn't have anything to comment on; I didn't
see anything that smelled fishy, or anything I'd want to improve.

I did have a few minor nits in 9/9, though.

Thanks.



--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v6 0/9] status: V2 porcelain status

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler 

This patch series adds porcelain V2 format to status.
This provides detailed information about file changes
and about the current branch.

The new output is accessed via:
git status --porcelain=v2 [--branch]

This v6 patch series addresses the unit test discussion
from the previous series.

I also moved the "else {" onto the previous line as
was discussed in a previous comment.


Jeff Hostetler (9):
  status: rename long-format print routines
  status: cleanup API to wt_status_print
  status: support --porcelain[=]
  status: collect per-file data for --porcelain=v2
  status: print per-file porcelain v2 status data
  status: print branch info with --porcelain=v2 --branch
  git-status.txt: describe --porcelain=v2 format
  test-lib-functions.sh: Add lf_to_nul
  status: unit tests for --porcelain=v2

 Documentation/git-status.txt | 133 +-
 builtin/commit.c |  78 +++---
 t/t7060-wtstatus.sh  |  21 ++
 t/t7064-wtstatus-pv2.sh  | 576 +++
 t/test-lib-functions.sh  |   4 +
 wt-status.c  | 570 +-
 wt-status.h  |  19 +-
 7 files changed, 1289 insertions(+), 112 deletions(-)
 create mode 100755 t/t7064-wtstatus-pv2.sh

-- 
2.8.0.rc4.17.gac42084.dirty

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html