[PATCH] Add very verbose porcelain output to status

2016-07-07 Thread Jeff Hostetler
this mode to offer more information. Just like we do elsewhere in Git's source code, we now interpret multiple `--verbose` flags accumulatively, and show substantially more information in porcelain mode at verbosity level 2. Signed-off-by: Jeff Hostetler --- Documentation/git-status

Re: [PATCH] Add very verbose porcelain output to status

2016-07-12 Thread Jeff Hostetler
lines and see how it looks. Thank again, Jeff On 07/12/2016 11:07 AM, Jeff King wrote: On Thu, Jul 07, 2016 at 03:26:28PM -0400, Jeff Hostetler wrote: Tools interacting with Git repositories may need to know the complete state of the working directory. For efficiency, it would be good to

[PATCH v1 4/6] Expanded branch header for Porcelain Status V2

2016-07-19 Thread Jeff Hostetler
. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt-status.c | 89 wt-status.h | 2 ++ 3 files changed, 96 insertions(+) diff --git a/builtin/commit.c b/builtin/commit.c index b5ec9b9..830f688 100644 --- a/builtin

[PATCH v1 1/6] Allow --porcelain[=] in status and commit commands

2016-07-19 Thread Jeff Hostetler
Update the --porcelain argument to take an optional version number. This will allow us to define new porcelain formats in the future. This default to 1 and represents the existing porcelain format. Signed-off-by: Jeff Hostetler --- Documentation/git-commit.txt | 2 +- Documentation/git

[PATCH v1 5/6] Add porcelain V2 documentation to status manpage

2016-07-19 Thread Jeff Hostetler
This commit updates the status manpage to include information about porcelain format V2. Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 62 +--- 1 file changed, 59 insertions(+), 3 deletions(-) diff --git a/Documentation/git-status.txt

[PATCH v1 3/6] Per-file output for Porcelain Status V2

2016-07-19 Thread Jeff Hostetler
values reflect the stage 1, 2, and 3 values. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 9 ++ wt-status.c | 398 ++- wt-status.h | 13 ++ 3 files changed, 419 insertions(+), 1 deletion(-) diff --git a/builtin/commit.c b

[PATCH v1 0/6] Porcelain Status V2

2016-07-19 Thread Jeff Hostetler
could be added to both formats. Jeff Hostetler (6): Allow --porcelain[=] in status and commit commands Status and checkout unit tests for --porcelain[=] Per-file output for Porcelain Status V2 Expanded branch header for Porcelain Status V2 Add porcelain V2 documentation to status manpage Unit

[PATCH v1 6/6] Unit tests for V2 porcelain status

2016-07-19 Thread Jeff Hostetler
This commit contains unit tests to exercise the V2 porcelain status format. Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 461 t/t7501-commit.sh | 9 + 2 files changed, 470 insertions(+) create mode 100755 t/t7064-wtstatus

[PATCH v1 2/6] Status and checkout unit tests for --porcelain[=]

2016-07-19 Thread Jeff Hostetler
Simple unit tests to validate the argument parsing. Signed-off-by: Jeff Hostetler --- t/t7060-wtstatus.sh | 21 + t/t7501-commit.sh | 14 ++ 2 files changed, 35 insertions(+) diff --git a/t/t7060-wtstatus.sh b/t/t7060-wtstatus.sh index 44bf1d8..a39b0e2 100755

Re: [PATCH v1 1/6] Allow --porcelain[=] in status and commit commands

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 11:08 AM, Johannes Schindelin wrote: On Tue, 19 Jul 2016, Jeff Hostetler wrote: diff --git a/builtin/commit.c b/builtin/commit.c + } else if (arg) { + int n = strtol(arg, NULL, 10); + if (n == 1) + *value

Re: [PATCH v1 5/6] Add porcelain V2 documentation to status manpage

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 11:29 AM, Jakub Narębski wrote: W dniu 2016-07-20 o 00:10, Jeff Hostetler pisze: +Porcelain Format Version 2 +~~ + + +If `--branch` is given, a header line showing branch tracking information +is printed. This line begins with "### branch: "

Re: [PATCH v1 6/6] Unit tests for V2 porcelain status

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 11:30 AM, Jakub Narębski wrote: W dniu 2016-07-20 o 00:10, Jeff Hostetler pisze: +test_expect_success pre_initial_commit_0 ' + printf "## branch: (initial) master\n" >expected && + printf "?? actual\n" >>expected

Re: [PATCH v1 2/6] Status and checkout unit tests for --porcelain[=]

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 11:19 AM, Johannes Schindelin wrote: Hi Jeff, On Tue, 19 Jul 2016, Jeff Hostetler wrote: Simple unit tests to validate the argument parsing. Signed-off-by: Jeff Hostetler They are simple alright, but do we really need so many of them? I would like to keep the ones in t7060

Re: [PATCH v1 1/6] Allow --porcelain[=] in status and commit commands

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 11:58 AM, Jeff King wrote: On Tue, Jul 19, 2016 at 06:10:53PM -0400, Jeff Hostetler wrote: +static int opt_parse_porcelain(const struct option *opt, const char *arg, int unset) +{ + enum wt_status_format *value = (enum wt_status_format *)opt->value; + if (un

Re: [PATCH v1 2/6] Status and checkout unit tests for --porcelain[=]

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 12:00 PM, Jeff King wrote: On Tue, Jul 19, 2016 at 06:10:54PM -0400, Jeff Hostetler wrote: +test_expect_failure '--porcelain=bogus with stuff to commit returns ok' ' + echo bongo bongo bongo >>file && + git commit -m next -a --porcela

Re: [PATCH v1 2/6] Status and checkout unit tests for --porcelain[=]

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 12:03 PM, Jeff King wrote: On Wed, Jul 20, 2016 at 10:00:07AM -0600, Jeff King wrote: On Tue, Jul 19, 2016 at 06:10:54PM -0400, Jeff Hostetler wrote: +test_expect_failure '--porcelain=bogus with stuff to commit returns ok' ' + echo bongo bon

Re: [PATCH v1 4/6] Expanded branch header for Porcelain Status V2

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 12:06 PM, Jeff King wrote: On Tue, Jul 19, 2016 at 06:10:56PM -0400, Jeff Hostetler wrote: + } else { + /* +* TODO All of various print routines allow for s->branch to be null. +* TODO When can this happen and what should

Re: [PATCH v1 0/6] Porcelain Status V2

2016-07-20 Thread Jeff Hostetler
On 07/20/2016 12:15 PM, Jeff King wrote: One final bit of food for thought. Just yesterday somebody asked me about renewing the old idea of using a more standardized format for machine-readable output, like --json. That's obviously something that would exist alongside the existing formats for

Re: [PATCH v1 3/6] Per-file output for Porcelain Status V2

2016-07-21 Thread Jeff Hostetler
On 07/20/2016 05:31 PM, Junio C Hamano wrote: The code seems to assume that d->porcelain_v2.* fields are initialized earlier in the callchain to reasonable values (e.g. STATUS_ADDED case does not clear .mode_head to "missing"); I am not sure if that is easier to read or fill in all the values

Re: [PATCH v1 4/6] Expanded branch header for Porcelain Status V2

2016-07-21 Thread Jeff Hostetler
On 07/21/2016 11:46 AM, Johannes Schindelin wrote: On Wed, 20 Jul 2016, Jeff King wrote: On Wed, Jul 20, 2016 at 02:20:24PM -0400, Jeff Hostetler wrote: IIRC, it happens when HEAD points to a broken ref. So something like: git init echo broken >.git/refs/heads/master would ca

[PATCH v2 0/8] status: V2 porcelain status

2016-07-25 Thread Jeff Hostetler
es are now completely separate and have a unique prefix key (and are grouped by type). The unit tests have been converted to use heredoc's. I removed the v2 argument from git commit --porcelain since it didn't really fit here. Jeff Hostetler (8): status: rename long-format print routines st

[PATCH v2 7/8] status: update git-status.txt for --porcelain=v2

2016-07-25 Thread Jeff Hostetler
Update status manpage to include information about porcelain v2 format. Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 83 ++-- 1 file changed, 80 insertions(+), 3 deletions(-) diff --git a/Documentation/git-status.txt b/Documentation

[PATCH v2 6/8] status: print branch info with --porcelain=v2 --branch

2016-07-25 Thread Jeff Hostetler
Expand porcelain v2 output to include branch and tracking branch information. This includes the commit SHA, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt-status.c | 86

[PATCH v2 4/8] status: per-file data collection for --porcelain=v2

2016-07-25 Thread Jeff Hostetler
, a GUI IDE might need the file mode to display the correct icon for a changed item. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 3 ++ wt-status.c | 114 ++- wt-status.h | 17 + 3 files changed, 133 insertions(+), 1

[PATCH v2 2/8] status: cleanup API to wt_status_print

2016-07-25 Thread Jeff Hostetler
formats and isolate that within wt-status.c Signed-off-by: Jeff Hostetler --- builtin/commit.c | 51 +-- wt-status.c | 25 ++--- wt-status.h | 16 3 files changed, 43 insertions(+), 49 deletions(-) diff

[PATCH v2 3/8] status: support --porcelain[=]

2016-07-25 Thread Jeff Hostetler
Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. Signed-off-by: Jeff Hostetler --- Documentation/git-statu

[PATCH v2 8/8] status: tests for --porcelain=v2

2016-07-25 Thread Jeff Hostetler
Unit tests for porcelain v2 status format. Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 542 1 file changed, 542 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff --git a/t/t7064-wtstatus-pv2.sh b/t/t7064-wtstatus-pv2

[PATCH v2 5/8] status: print per-file porcelain v2 status data

2016-07-25 Thread Jeff Hostetler
Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler --- wt-status.c | 285 +++- 1 file changed, 284 insertions(+), 1 deletion(-) diff --git a/wt-status.c b/wt-status.c index 54aedc1..ffdfe11 100644 --- a/wt

[PATCH v2 1/8] status: rename long-format print routines

2016-07-25 Thread Jeff Hostetler
Renamed the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output. This will hopefully reduce confusion as other status formats are added in the future. Signed-off-by: Jeff Hostetler

Re: [PATCH v2 4/8] status: per-file data collection for --porcelain=v2

2016-07-26 Thread Jeff Hostetler
On 07/25/2016 04:14 PM, Junio C Hamano wrote: Jeff Hostetler writes: +static void aux_updated_entry_porcelain_v2( + struct wt_status *s, + struct wt_status_change_data *d, + struct diff_filepair *p) +{ + switch (p->status) { + case DIFF_STATUS_AD

Re: [PATCH v2 5/8] status: print per-file porcelain v2 status data

2016-07-26 Thread Jeff Hostetler
On 07/25/2016 04:23 PM, Junio C Hamano wrote: Jeff Hostetler writes: +static void wt_porcelain_v2_print(struct wt_status *s); + There is no point in this forward declaration, if you just place the implementation of these functions here, no? Right. I just did it that way to make the

Re: [PATCH v2 7/8] status: update git-status.txt for --porcelain=v2

2016-07-26 Thread Jeff Hostetler
On 07/25/2016 06:43 PM, Jakub Narębski wrote: W dniu 2016-07-25 o 21:25, Jeff Hostetler pisze: +Porcelain Format Version 2 +~~ + +Version 2 format adds more detailed information about the state of +the worktree and the changed items. I think it should be &quo

[PATCH v3 3/8] status: support --porcelain[=]

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[PATCH v3 0/8] status: V2 porcelain status

2016-07-26 Thread Jeff Hostetler
, but silently also allow --porcelain=2. Jeff Hostetler (8): status: rename long-format print routines status: cleanup API to wt_status_print status: support --porcelain[=] status: per-file data collection for --porcelain=v2 status: print per-file porcelain v2 status data status: print branch

[PATCH v3 2/8] status: cleanup API to wt_status_print

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine and eliminate the switch statements from builtin/commit.c This will allow us to more

[PATCH v3 1/8] status: rename long-format print routines

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Renamed the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output. This will hopefully reduce confusion as other status formats are added in the future. Signed-off-by

[PATCH v3 5/8] status: print per-file porcelain v2 status data

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- wt-status.c | 283 +++- 1 file changed, 282 insertions(+), 1 deletion(-) diff --git a/wt-status.c b

[PATCH v3 7/8] status: update git-status.txt for --porcelain=v2

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Update status manpage to include information about porcelain v2 format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 93 ++-- 1 file changed, 90 insertions(+), 3 deletions(-) diff

[PATCH v3 8/8] status: tests for --porcelain=v2

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Unit tests for porcelain v2 status format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 585 1 file changed, 585 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff

[PATCH v3 6/8] status: print branch info with --porcelain=v2 --branch

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler Expand porcelain v2 output to include branch and tracking branch information. This includes the commit SHA, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt

[PATCH v3 4/8] status: per-file data collection for --porcelain=v2

2016-07-26 Thread Jeff Hostetler
From: Jeff Hostetler The output of `git status --porcelain` leaves out many details about the current status that clients might like to have. This can force them to be less efficient as they may need to launch secondary commands (and try to match the logic within git) to accumulate this extra

Re: [PATCH v3 7/8] status: update git-status.txt for --porcelain=v2

2016-08-01 Thread Jeff Hostetler
On 07/30/2016 01:22 PM, Jakub Narębski wrote: W dniu 26.07.2016 o 23:11, Jeff Hostetler pisze: This is a nice change, available because of lack of backward compatibility with v1. The porcelain v2 format branch-related information could be enhanced without risk of breaking parsers, or having

[PATCH v4 8/8] status: tests for --porcelain=v2

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Unit tests for porcelain v2 status format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 585 1 file changed, 585 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff

[PATCH v4 0/8] status: V2 porcelain status

2016-08-02 Thread Jeff Hostetler
ation/git-status.txt for clarity. Jeff Hostetler (8): status: rename long-format print routines status: cleanup API to wt_status_print status: support --porcelain[=] status: per-file data collection for --porcelain=v2 status: print per-file porcelain v2 status data status: print branch

[PATCH v4 2/8] status: cleanup API to wt_status_print

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine and eliminate the switch statements from builtin/commit.c This will allow us to more

[PATCH v4 7/8] git-status.txt: describe --porcelain=v2 format

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Update status manpage to include information about porcelain V2 format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 123 +-- 1 file changed, 119 insertions(+), 4 deletions(-) diff

[PATCH v4 3/8] status: support --porcelain[=]

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[PATCH v4 1/8] status: rename long-format print routines

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Renamed the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output. This will hopefully reduce confusion as other status formats are added in the future. Signed-off-by

[PATCH v4 5/8] status: print per-file porcelain v2 status data

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- wt-status.c | 283 +++- 1 file changed, 282 insertions(+), 1 deletion(-) diff --git a/wt-status.c b

[PATCH v4 6/8] status: print branch info with --porcelain=v2 --branch

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Expand porcelain v2 output to include branch and tracking branch information. This includes the commit SHA, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt

[PATCH v4 4/8] status: per-file data collection for --porcelain=v2

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler The output of `git status --porcelain` leaves out many details about the current status that clients might like to have. This can force them to be less efficient as they may need to launch secondary commands (and try to match the logic within git) to accumulate this extra

Re: [PATCH v3 7/8] status: update git-status.txt for --porcelain=v2

2016-08-02 Thread Jeff Hostetler
On 08/02/2016 11:19 AM, Jakub Narębski wrote: W dniu 01.08.2016 o 17:39, Jeff Hostetler pisze: On 07/30/2016 01:22 PM, Jakub Narębski wrote: W dniu 26.07.2016 o 23:11, Jeff Hostetler pisze: This is a nice change, available because of lack of backward compatibility with v1. The porcelain v2

Re: [PATCH v4 1/8] status: rename long-format print routines

2016-08-04 Thread Jeff Hostetler
On 08/03/2016 05:28 PM, Junio C Hamano wrote: Signed-off-by: Jeff Hostetler Signed-off-by: Jeff Hostetler Hmm, are these physically the same people? If so, which one do you want to be known as? Yes, these are both my addresses. Still struggling a little with some SMTP issues. Please

Re: [PATCH v4 2/8] status: cleanup API to wt_status_print

2016-08-04 Thread Jeff Hostetler
On 08/03/2016 05:36 PM, Junio C Hamano wrote: Jeff Hostetler writes: From: Jeff Hostetler diff --git a/wt-status.h b/wt-status.h index 2023a3c..a859a12 100644 --- a/wt-status.h +++ b/wt-status.h @@ -43,6 +43,15 @@ struct wt_status_change_data { unsigned new_submodule_commits : 1

Re: [PATCH v4 6/8] status: print branch info with --porcelain=v2 --branch

2016-08-05 Thread Jeff Hostetler
On 08/05/2016 01:01 PM, Junio C Hamano wrote: Jeff Hostetler writes: /* + * Print branch information for porcelain v2 output. These lines + * are printed when the '--branch' parameter is given. + * + *# branch.oid + *# branch.head Just bikeshedding, but ... +

Re: [PATCH v4 7/8] git-status.txt: describe --porcelain=v2 format

2016-08-05 Thread Jeff Hostetler
On 08/05/2016 01:50 PM, Junio C Hamano wrote: Jeff Hostetler writes: +Porcelain Format Version 2 +~~ + +Version 2 format adds more detailed information about the state of +the worktree and changed items. Version 2 also defines an extensible +set of easy to parse

Re: [PATCH v4 8/8] status: tests for --porcelain=v2

2016-08-05 Thread Jeff Hostetler
On 08/05/2016 02:12 PM, Junio C Hamano wrote: Jeff Hostetler writes: +## +## Confirm output prior to initial commit. +## + +test_expect_success

Re: [PATCH] wt-status.c: mark a file-local symbol as static

2016-08-05 Thread Jeff Hostetler
On 08/05/2016 04:55 PM, Ramsay Jones wrote: Signed-off-by: Ramsay Jones --- Hi Jeff, If you need to re-roll your 'jh/status-v2-porcelain' branch, could you please squash this into the relevant patch (37f7104f, "status: print per-file porcelain v2 status data", 02-08-2016). Thanks! ATB, Ra

Re: [PATCH v4 5/8] status: print per-file porcelain v2 status data

2016-08-05 Thread Jeff Hostetler
On 08/05/2016 05:02 PM, Jeff King wrote: On Tue, Aug 02, 2016 at 10:12:14AM -0400, Jeff Hostetler wrote: +static void wt_porcelain_v2_print_unmerged_entry( + struct string_list_item *it, + struct wt_status *s) +{ + struct wt_status_change_data *d = it->util; + co

[PATCH v5 2/9] status: cleanup API to wt_status_print

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide the details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine. Eliminate the switch statements from builtin/commit.c. Allow details of new status

[PATCH v5 6/9] status: print branch info with --porcelain=v2 --branch

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Expand porcelain v2 output to include branch and tracking branch information. This includes the commit id, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt-status.c | 90

[PATCH v5 9/9] status: unit tests for --porcelain=v2

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Test porcelain v2 status format. Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 597 1 file changed, 597 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff --git a/t/t7064-wtstatus-pv2.sh b/t/t7064

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

2016-08-05 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 v5 patch series addresses all comments received today

[PATCH v5 8/9] test-lib-functions.sh: Add lf_to_nul

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Add lf_to_nul() function to test-lib-functions. Signed-off-by: Jeff Hostetler --- t/test-lib-functions.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 4f7eadb..fdaeb3a 100644 --- a/t/test-lib-functions.sh

[PATCH v5 5/9] status: print per-file porcelain v2 status data

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler --- wt-status.c | 285 +++- 1 file changed, 284 insertions(+), 1 deletion(-) diff --git a/wt-status.c b/wt-status.c index ca8023e

[PATCH v5 1/9] status: rename long-format print routines

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Rename the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output and reduce developer confusion as other status formats are added in the future. Signed-off-by: Jeff

[PATCH v5 3/9] status: support --porcelain[=]

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[PATCH v5 4/9] status: collect per-file data for --porcelain=v2

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Collect extra per-file data for porcelain V2 format. The output of `git status --porcelain` leaves out many details about the current status that clients might like to have. This can force them to be less efficient as they may need to launch secondary commands (and try to

[PATCH v5 7/9] git-status.txt: describe --porcelain=v2 format

2016-08-05 Thread Jeff Hostetler
From: Jeff Hostetler Update status manpage to include information about porcelain v2 format. Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 126 +-- 1 file changed, 122 insertions(+), 4 deletions(-) diff --git a/Documentation/git

Re: [PATCH v5 9/9] status: unit tests for --porcelain=v2

2016-08-10 Thread Jeff Hostetler
On 08/08/2016 01:07 PM, Junio C Hamano wrote: Jeff Hostetler writes: +test_expect_success pre_initial_commit_0 ' + ... + git status --porcelain=v2 --branch --untracked-files=normal >actual && + test_cmp expect actual +' + + +test_expect_success

Re: [PATCH v5 9/9] status: unit tests for --porcelain=v2

2016-08-10 Thread Jeff Hostetler
On 08/10/2016 06:41 PM, Junio C Hamano wrote: Jeff Hostetler writes: Having said all that, it is OK to fix their titles after the current 9-patch series lands on 'next'; incremental refinements are easier on reviewers than having to review too many rerolls. I'll change the

Re: Small regression on Windows

2017-02-13 Thread Jeff Hostetler
a9b8a09c3c30886c79133da9f48ef9f98c21c3b2 is the first bad commit commit a9b8a09c3c30886c79133da9f48ef9f98c21c3b2 Author: Jeff Hostetler Date: Thu Dec 22 18:09:23 2016 +0100 mingw: replace isatty() hack Git for Windows has carried a patch that depended on internals of MSVC runtime, but it does

Re: [PATCH 0/5] A series of performance enhancements in the memihash and name-cache area

2017-02-15 Thread Jeff Hostetler
On 2/14/2017 5:03 PM, Jeff King wrote: On Tue, Feb 14, 2017 at 12:31:46PM +0100, Johannes Schindelin wrote: On Windows, calls to memihash() and maintaining the istate.name_hash and istate.dir_hash HashMaps take significant time on very large repositories. This series of changes reduces the ov

RE: [PATCH 0/5] A series of performance enhancements in the memihash and name-cache area

2017-02-18 Thread Jeff Hostetler
> Jeff King writes: > >> On Fri, Feb 17, 2017 at 09:58:21PM -0800, Junio C Hamano wrote: >> >>> Jeff Hostetler writes: >>> >>> > I'll try to put together a before/after perf-test to better >>> > demonstrate this. >>

RE: [PATCH 0/5] A series of performance enhancements in the memihash and name-cache area

2017-02-18 Thread Jeff Hostetler
From: Junio C Hamano [mailto:jch2...@gmail.com] On Behalf Of Junio C Hamano > Jeff King writes: >> On Wed, Feb 15, 2017 at 09:27:53AM -0500, Jeff Hostetler wrote: >> >>> I have some informal numbers in a spreadsheet. I was seeing >>> a 8-9% speed u

RE: [PATCH 3/5] name-hash: precompute hash values during preload-index

2017-02-18 Thread Jeff Hostetler
From: Junio C Hamano [mailto:jch2...@gmail.com] On Behalf Of Junio C Hamano > > The fact that each preload_thread() still walks the index in-order > makes me wonder if it may allow us to further optimize the "dir" > part of the hash by passing the previous ce for which we already > precomputed h

[PATCH v6 3/9] status: support --porcelain[=]

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[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

[PATCH v6 6/9] status: print branch info with --porcelain=v2 --branch

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Expand porcelain v2 output to include branch and tracking branch information. This includes the commit id, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt-status.c | 90

[PATCH v6 2/9] status: cleanup API to wt_status_print

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide the details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine. Eliminate the switch statements from builtin/commit.c. Allow details of new status

[PATCH v6 9/9] status: unit tests for --porcelain=v2

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Test porcelain v2 status format. Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 576 1 file changed, 576 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff --git a/t/t7064-wtstatus-pv2.sh b/t/t7064

[PATCH v6 8/9] test-lib-functions.sh: Add lf_to_nul

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Add lf_to_nul() function to test-lib-functions. Signed-off-by: Jeff Hostetler --- t/test-lib-functions.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 4f7eadb..fdaeb3a 100644 --- a/t/test-lib-functions.sh

[PATCH v6 7/9] git-status.txt: describe --porcelain=v2 format

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Update status manpage to include information about porcelain v2 format. Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 126 +-- 1 file changed, 122 insertions(+), 4 deletions(-) diff --git a/Documentation/git

[PATCH v6 4/9] status: collect per-file data for --porcelain=v2

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Collect extra per-file data for porcelain V2 format. The output of `git status --porcelain` leaves out many details about the current status that clients might like to have. This can force them to be less efficient as they may need to launch secondary commands (and try to

[PATCH v6 5/9] status: print per-file porcelain v2 status data

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler --- wt-status.c | 285 +++- 1 file changed, 284 insertions(+), 1 deletion(-) diff --git a/wt-status.c b/wt-status.c index 904b5c2

[PATCH v6 1/9] status: rename long-format print routines

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Rename the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output and reduce developer confusion as other status formats are added in the future. Signed-off-by: Jeff

[PATCH v7 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 v7 patch series address the most recent feedback on the

[PATCH v7 8/9] test-lib-functions.sh: Add lf_to_nul

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Add lf_to_nul() function to test-lib-functions. Signed-off-by: Jeff Hostetler --- t/test-lib-functions.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 4f7eadb..fdaeb3a 100644 --- a/t/test-lib-functions.sh

[PATCH v7 2/9] status: cleanup API to wt_status_print

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide the details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine. Eliminate the switch statements from builtin/commit.c. Allow details of new status

[PATCH v7 9/9] status: unit tests for --porcelain=v2

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Test porcelain v2 status format. Signed-off-by: Jeff Hostetler --- t/t7064-wtstatus-pv2.sh | 592 1 file changed, 592 insertions(+) create mode 100755 t/t7064-wtstatus-pv2.sh diff --git a/t/t7064-wtstatus-pv2.sh b/t/t7064

[PATCH v7 4/9] status: collect per-file data for --porcelain=v2

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Collect extra per-file data for porcelain V2 format. The output of `git status --porcelain` leaves out many details about the current status that clients might like to have. This can force them to be less efficient as they may need to launch secondary commands (and try to

[PATCH v7 6/9] status: print branch info with --porcelain=v2 --branch

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Expand porcelain v2 output to include branch and tracking branch information. This includes the commit id, the branch, the upstream branch, and the ahead and behind counts. Signed-off-by: Jeff Hostetler --- builtin/commit.c | 5 wt-status.c | 90

[PATCH v7 5/9] status: print per-file porcelain v2 status data

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Print per-file information in porcelain v2 format. Signed-off-by: Jeff Hostetler --- wt-status.c | 285 +++- 1 file changed, 284 insertions(+), 1 deletion(-) diff --git a/wt-status.c b/wt-status.c index aa804b5

[PATCH v7 7/9] git-status.txt: describe --porcelain=v2 format

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Update status manpage to include information about porcelain v2 format. Signed-off-by: Jeff Hostetler --- Documentation/git-status.txt | 126 +-- 1 file changed, 122 insertions(+), 4 deletions(-) diff --git a/Documentation/git

[PATCH v7 3/9] status: support --porcelain[=]

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[PATCH v7 1/9] status: rename long-format print routines

2016-08-11 Thread Jeff Hostetler
From: Jeff Hostetler Rename the various wt_status_print*() routines to be wt_longstatus_print*() to make it clear that these routines are only concerned with the normal/long status output and reduce developer confusion as other status formats are added in the future. Signed-off-by: Jeff

Re: [PATCH v6 9/9] status: unit tests for --porcelain=v2

2016-08-11 Thread Jeff Hostetler
On 08/11/2016 02:36 PM, Junio C Hamano wrote: Jeff Hostetler writes: From: Jeff Hostetler +. ./test-lib.sh + +OID_EMPTY=e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 It seems that test-lib.sh these days has EMPTY_BLOB defined for your use. You can remove this and replace its use (just two

[PATCH v8 8/9] test-lib-functions.sh: Add lf_to_nul

2016-08-12 Thread Jeff Hostetler
From: Jeff Hostetler Add lf_to_nul() function to test-lib-functions. Signed-off-by: Jeff Hostetler --- t/test-lib-functions.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 4f7eadb..fdaeb3a 100644 --- a/t/test-lib-functions.sh

[PATCH v8 3/9] status: support --porcelain[=]

2016-08-12 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.)

[PATCH v8 2/9] status: cleanup API to wt_status_print

2016-08-12 Thread Jeff Hostetler
From: Jeff Hostetler Refactor the API between builtin/commit.c and wt-status.[ch]. Hide the details of the various wt_*status_print() routines inside wt-status.c behind a single (new) wt_status_print() routine. Eliminate the switch statements from builtin/commit.c. Allow details of new status

  1   2   3   4   5   6   7   8   9   10   >