[PATCH v4 5/6] midx: honor the MIDX_PROGRESS flag in midx_repack

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Update midx_repack to only display progress when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/midx.c b/midx.c index 35bdc5f72c..b5f7f1c54a 100644 --- a/midx.c +++ b/midx.c @@ -1373,6

[PATCH v4 3/6] midx: add progress to expire_midx_packs

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Add progress to expire_midx_packs. Progress is displayed when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 12 1 file changed, 12 insertions(+) diff --git a/midx.c b/midx.c index 006f36b570..7157259ca2 100644 --- a/midx.c +++ b

[PATCH v4 2/6] midx: add progress to write_midx_file

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Add progress to write_midx_file. Progress is displayed when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/midx.c b/midx.c index f169a681dd..006f36b570

[PATCH v4 4/6] midx: honor the MIDX_PROGRESS flag in verify_midx_file

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Update verify_midx_file to only display progress when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/midx.c b/midx.c index 7157259ca2..35bdc5f72c 100644 --- a

[PATCH v4 6/6] multi-pack-index: add [--[no-]progress] option.

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Add the --[no-]progress option to git multi-pack-index. Pass the MIDX_PROGRESS flag to the subcommand functions when progress should be displayed by multi-pack-index. The progress feature was added to 'verify' in 144d703 ("multi-pack-index: report progress

[PATCH v4 1/6] midx: add MIDX_PROGRESS flag

2019-10-21 Thread William Baker via GitGitGadget
From: William Baker Add the MIDX_PROGRESS flag and update the write|verify|expire|repack functions in midx.h to accept a flags parameter. The MIDX_PROGRESS flag indicates whether the caller of the function would like progress information to be displayed. This patch only changes the method

[PATCH v4 0/6] multi-pack-index: add --no-progress

2019-10-21 Thread William Baker via GitGitGadget
patch. Thanks, William William Baker (6): midx: add MIDX_PROGRESS flag midx: add progress to write_midx_file midx: add progress to expire_midx_packs midx: honor the MIDX_PROGRESS flag in verify_midx_file midx: honor the MIDX_PROGRESS flag in midx_repack multi-pack-index: add [--[no-]p

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-10-18 Thread William Baker
On 10/16/19 12:48 PM, William Baker wrote:>> I do not care too deeply either way, but if you wrote it in one way, >> how about completing the series without changing it in the middle, >> and leave the clean-ups to a follow-up series (if needed)? > > > That plan s

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-10-16 Thread William Baker
On 10/15/19 7:09 PM, Junio C Hamano wrote: >> At this time there are no other MIDX_* flags and so there's always the option >> to revisit the best way to handle multiple MIDX_* flags if/when additional >> flags are added. > > I do not care too deeply either way, but if you wrote it in one way, > h

[PATCH 1/1] t7519-status-fsmonitor: improve comments

2019-10-16 Thread William Baker via GitGitGadget
From: William Baker The comments for the staging/unstaging test did not accurately describe the scenario being tested. It is not essential that the test files being staged/unstaged appear at the end of the index. All that is required is that the test files are not flagged with

[PATCH 0/1] t7519-status-fsmonitor: improve comments

2019-10-16 Thread William Baker via GitGitGadget
This patch series is based on the 'wb/fsmonitor-bitmap-fix' branch and addresses feedback on the new test that was added there. I've updated the comments for the new test to more accurately describe the scenario being tested. Thanks, William William Baker (1): t7519-s

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-10-15 Thread William Baker
On 10/8/19 6:32 PM, SZEDER Gábor wrote: >>> No, they tend to show (PROGRESS | REGRESS), at least both gdb and lldb >>> do. > > I was wrong here, gdb does this, but lldb, unfortunately, doesn't; see > my other reply in this thread. > >> What I was worried about is that the constants that are used t

Re: [PATCH v3 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-15 Thread William Baker
On 10/11/19 6:26 PM, Junio C Hamano wrote: > "William Baker via GitGitGadget" writes: > >> +# Test staging/unstaging files that appear at the end of the index. Test >> +# file names begin with 'z' so that they are sorted to the end of the >> index.

[PATCH v3 0/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-11 Thread William Baker via GitGitGadget
Dscho. Thanks, William William Baker (1): fsmonitor: don't fill bitmap with entries to be removed fsmonitor.c | 29 - t/t7519-status-fsmonitor.sh | 17 + t/t7519/fsmonitor-env | 24 3 files change

[PATCH v3 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-11 Thread William Baker via GitGitGadget
From: William Baker While doing some testing with fsmonitor enabled I found that git commands would segfault after staging and unstaging an untracked file. Looking at the crash it appeared that fsmonitor_ewah_callback was attempting to adjust bits beyond the bounds of the index cache. Digging

Re: [PATCH v2 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-11 Thread William Baker
On 10/10/19 4:22 AM, SZEDER Gábor wrote: >>> +# This test covers staging/unstaging files that appear at the end of the >>> index. >>> +# Test files with names beginning with 'z' are used under the assumption >>> that >>> +# earlier tests do not add/leave index entries that sort below them. > >

[PATCH v2 0/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-09 Thread William Baker via GitGitGadget
ment to be more descriptive Thanks, William William Baker (1): fsmonitor: don't fill bitmap with entries to be removed fsmonitor.c | 29 - t/t7519-status-fsmonitor.sh | 13 + t/t7519/fsmonitor-env | 24 +

[PATCH v2 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-09 Thread William Baker via GitGitGadget
From: William Baker While doing some testing with fsmonitor enabled I found that git commands would segfault after staging and unstaging an untracked file. Looking at the crash it appeared that fsmonitor_ewah_callback was attempting to adjust bits beyond the bounds of the index cache. Digging

Re: Raise your hand to Ack jk/code-of-conduct if your Ack fell thru cracks

2019-10-09 Thread William Baker
but I'd still wait for a few > days for people who expressed their Acks but your scan missed, or > those who wanted to give their Acks but forgot to do so, to raise > their hands on this thread. > This looks good to me as well: Acked-by: William Baker Thanks, William

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-10-08 Thread William Baker
On 10/7/19 9:30 PM, Junio C Hamano wrote: > SZEDER Gábor writes: > >>> func(PROGRESS | REGRESS); >>> func(PROGRESS + REGRESS); >>> func(PROGRESS * 3); >>> } >>> >>> how caller came about to give 3? >> >> No, they tend to show (PROGRESS | REGRESS), at least

Re: [PATCH 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-07 Thread William Baker
On 10/3/19 4:36 PM, Junio C Hamano wrote: >> +if (pos >= istate->cache_nr) >> +BUG("fsmonitor_dirty has more entries than the index >> (%"PRIuMAX" >= %"PRIuMAX")", >> +(uintmax_t)pos, (uintmax_t)istate->cache_nr); > > This is how we show size_t values without usin

[PATCH 1/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker While doing some testing with fsmonitor enabled I found that git commands would segfault after staging and unstaging an untracked file. Looking at the crash it appeared that fsmonitor_ewah_callback was attempting to adjust bits beyond the bounds of the index cache. Digging

[PATCH 0/1] fsmonitor: don't fill bitmap with entries to be removed

2019-10-03 Thread William Baker via GitGitGadget
some BUG checks that would have helped catch this problem sooner. Special thanks to Dscho for pointing me in the right direction and suggesting a test that can reproduce the issue. Thanks, William William Baker (1): fsmonitor: don't fill bitmap with entries to be removed f

[PATCH v3 2/6] midx: add progress to write_midx_file

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Add progress to write_midx_file. Progress is displayed when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/midx.c b/midx.c index b2673f52e8..0808a40dd4

[PATCH v3 5/6] midx: honor the MIDX_PROGRESS flag in midx_repack

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Update midx_repack to only display progress when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/midx.c b/midx.c index ced5898bbf..1c5ddeb007 100644 --- a/midx.c +++ b/midx.c @@ -1374,6

[PATCH v3 6/6] multi-pack-index: add [--[no-]progress] option.

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Add the --[no-]progress option to git multi-pack-index. Pass the MIDX_PROGRESS flag to the subcommand functions when progress should be displayed by multi-pack-index. The progress feature was added to 'verify' in 144d703 ("multi-pack-index: report progress

[PATCH v3 1/6] midx: add MIDX_PROGRESS flag

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Add the MIDX_PROGRESS flag and update the write|verify|expire|repack functions in midx.h to accept a flags parameter. The MIDX_PROGRESS flag indicates whether the caller of the function would like progress information to be displayed. This patch only changes the method

[PATCH v3 4/6] midx: honor the MIDX_PROGRESS flag in verify_midx_file

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Update verify_midx_file to only display progress when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/midx.c b/midx.c index f14bebb092..ced5898bbf 100644 --- a

[PATCH v3 3/6] midx: add progress to expire_midx_packs

2019-10-03 Thread William Baker via GitGitGadget
From: William Baker Add progress to expire_midx_packs. Progress is displayed when the MIDX_PROGRESS flag is set. Signed-off-by: William Baker --- midx.c | 12 1 file changed, 12 insertions(+) diff --git a/midx.c b/midx.c index 0808a40dd4..f14bebb092 100644 --- a/midx.c +++ b

[PATCH v3 0/6] multi-pack-index: add --no-progress

2019-10-03 Thread William Baker via GitGitGadget
ed Thanks, William William Baker (6): midx: add MIDX_PROGRESS flag midx: add progress to write_midx_file midx: add progress to expire_midx_packs midx: honor the MIDX_PROGRESS flag in verify_midx_file midx: honor the MIDX_PROGRESS flag in midx_repack multi-pack-index: add [--[no-]progre

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-10-02 Thread William Baker
On 10/2/19 12:04 AM, Junio C Hamano wrote>> >> Well, if you write >> >> enum { BIT_0 = 1, BIT_1 = 2, BIT_3 = 4 } var; >> >> it's pretty much a promise that the normal value for the var is one >> of these listed values to your readers. > > ... that is why compilers give a warning when you writ

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-09-30 Thread William Baker
On 9/27/19 8:40 PM, Junio C Hamano wrote: > SZEDER Gábor writes: > >>> +#define MIDX_PROGRESS (1 << 0) >> >> Please consider using an enum. > > If they are used by assiging one of their values, definitely a good > idea to use an enum. Are debuggers clever enough that they can > tell, when t

Re: [PATCH v2 2/6] midx: add progress to write_midx_file Add progress to write_midx_file. Progress is displayed when the MIDX_PROGRESS flag is set.

2019-09-30 Thread William Baker
On 9/27/19 8:49 PM, Junio C Hamano wrote: diff --git a/midx.c b/midx.c index b2673f52e8..54e4e93b2b 100644 --- a/midx.c +++ b/midx.c @@ -449,6 +449,8 @@ struct pack_list { uint32_t nr; uint32_t alloc; struct multi_pack_index *m; + struct progre

Re: [PATCH v2 1/6] midx: add MIDX_PROGRESS flag

2019-09-23 Thread William Baker
On 9/21/19 5:11 AM, SZEDER Gábor wrote: >> >> +#define MIDX_PROGRESS (1 << 0) > > Please consider using an enum. > > A preprocessor constant is just that: a number. It is shown as a > number while debugging, and there is no clear indication what related > values belong in the same group (a

Re: [PATCH v2 2/6] midx: add progress to write_midx_file Add progress to write_midx_file. Progress is displayed when the MIDX_PROGRESS flag is set.

2019-09-23 Thread William Baker
On 9/20/19 1:10 PM, Junio C Hamano wrote: >> diff --git a/midx.c b/midx.c >> index b2673f52e8..54e4e93b2b 100644 >> --- a/midx.c >> +++ b/midx.c >> @@ -449,6 +449,8 @@ struct pack_list { >> uint32_t nr; >> uint32_t alloc; >> struct multi_pack_index *m; >> +struct progress *progre

Re: [PATCH v2 0/6] multi-pack-index: add --no-progress

2019-09-20 Thread William Baker
On 9/20/19 1:23 PM, Philip Oakley wrote: > These subject lines are a bit long. Has there been some white space damage > during preparation, such that the end of line, and following blank line, for > a short subject line been compromised? > > Normally the 'git log --oneline' should fit within th

Re: [PATCH v2 0/6] multi-pack-index: add --no-progress

2019-09-20 Thread William Baker
On 9/20/19 12:54 PM, Junio C Hamano wrote: > > Do all of these commits have overly long title with no body recorded > in the commit objects, or is this just a mail sending program > screwing up? > Sorry about that! With some help from Stolee I found the issue was that I was missing a second lin

[PATCH v2 3/6] midx: add progress to expire_midx_packs Add progress to expire_midx_packs. Progress is displayed when the MIDX_PROGRESS flag is set.

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- midx.c | 12 1 file changed, 12 insertions(+) diff --git a/midx.c b/midx.c index 54e4e93b2b..3a0e161aea 100644 --- a/midx.c +++ b/midx.c @@ -1206,18 +1206,29 @@ int expire_midx_packs(struct repository *r, const char

[PATCH v2 6/6] multi-pack-index: add [--[no-]progress] option. Add the --[no-]progress option to git multi-pack-index. Pass the MIDX_PROGRESS flag to the subcommand functions when progress should be d

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- Documentation/git-multi-pack-index.txt | 6 ++- builtin/multi-pack-index.c | 18 +-- t/t5319-multi-pack-index.sh| 69 ++ 3 files changed, 87 insertions(+), 6 deletions(-) diff --git a

[PATCH v2 2/6] midx: add progress to write_midx_file Add progress to write_midx_file. Progress is displayed when the MIDX_PROGRESS flag is set.

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- midx.c | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/midx.c b/midx.c index b2673f52e8..54e4e93b2b 100644 --- a/midx.c +++ b/midx.c @@ -449,6 +449,8 @@ struct pack_list { uint32_t nr

[PATCH v2 4/6] midx: honor the MIDX_PROGRESS flag in verify_midx_file Update verify_midx_file to only display progress when the MIDX_PROGRESS flag is set.

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- midx.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/midx.c b/midx.c index 3a0e161aea..dc7c5f43f8 100644 --- a/midx.c +++ b/midx.c @@ -1098,15 +1098,16 @@ int verify_midx_file(struct repository *r

[PATCH v2 0/6] multi-pack-index: add --no-progress

2019-09-20 Thread William Baker via GitGitGadget
Hello Git contributors! My name is William Baker and I work at Microsoft. Over the past few years I've worked closely with the Microsoft team contributing to the git ecosystem and I'm excited to start working with the community. Derrick Stolee helped me pick out my first task, and i

[PATCH v2 5/6] midx: honor the MIDX_PROGRESS flag in midx_repack Update midx_repack to only display progress when the MIDX_PROGRESS flag is set.

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- midx.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/midx.c b/midx.c index dc7c5f43f8..106ccc4ab2 100644 --- a/midx.c +++ b/midx.c @@ -1374,6 +1374,12 @@ int midx_repack(struct repository *r, const char *object_dir, size_t

[PATCH v2 1/6] midx: add MIDX_PROGRESS flag Add the MIDX_PROGRESS flag and update the write|verify|expire|repack functions in midx.h to accept a flags parameter. The MIDX_PROGRESS flag indicates whet

2019-09-20 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- builtin/multi-pack-index.c | 8 builtin/repack.c | 2 +- midx.c | 8 midx.h | 10 ++ 4 files changed, 15 insertions(+), 13 deletions(-) diff --git a/builtin

Re: [PATCH 1/1] multi-pack-index: add --no-progress Add --no-progress option to git multi-pack-index. The progress feature was added in 144d703 ("multi-pack-index: report progress during 'verify'", 20

2019-09-16 Thread William Baker
On 9/13/19 1:26 PM, Junio C Hamano wrote: > Compare the ways how dispatching and command line option parsing of > subcommands in "multi-pack-index" and "commit-graph" are > implemented. When a command (e.g. "commit-graph") takes common > options and also has subcommands (e.g. "read" and "write")

Re: [PATCH 1/1] multi-pack-index: add --no-progress Add --no-progress option to git multi-pack-index. The progress feature was added in 144d703 ("multi-pack-index: report progress during 'verify'", 20

2019-09-13 Thread William Baker
Hi Junio, Thanks for the review! On 9/12/19 1:17 PM, Junio C Hamano wrote: >> +'git multi-pack-index' [--object-dir=] [--[no-]progress] > > I am wondering what the reasoning behind having this new one *after* > the subcommand while the existing one *before* is. Isn't the > --[no-]progress opt

[PATCH 0/1] multi-pack-index: add --no-progress

2019-09-11 Thread William Baker via GitGitGadget
Hello Git contributors! My name is William Baker and I work at Microsoft. Over the past few years I've worked closely with the Microsoft team contributing to the git ecosystem and I'm excited to start working with the community. Derrick Stolee helped me pick out my first task, and i

[PATCH 1/1] multi-pack-index: add --no-progress Add --no-progress option to git multi-pack-index. The progress feature was added in 144d703 ("multi-pack-index: report progress during 'verify'", 2018-0

2019-09-11 Thread William Baker via GitGitGadget
From: William Baker Signed-off-by: William Baker --- Documentation/git-multi-pack-index.txt | 6 +- builtin/multi-pack-index.c | 14 +--- midx.c | 30 +- midx.h | 6 -- t/t5319

[no subject]

2019-08-20 Thread William Baker
subscribe git