* [PATCH] doc/technical: remove outdated MIDX default note @ 2021-09-22 22:13 Eric Wong 2021-09-23 13:47 ` Derrick Stolee 0 siblings, 1 reply; 9+ messages in thread From: Eric Wong @ 2021-09-22 22:13 UTC (permalink / raw) To: git; +Cc: Derrick Stolee, Junio C Hamano Multi-pack-index files are used by default since commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the outdated note was misleading. Signed-off-by: Eric Wong <e@80x24.org> --- Documentation/technical/multi-pack-index.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/Documentation/technical/multi-pack-index.txt b/Documentation/technical/multi-pack-index.txt index 1a73c3ee20..2038fb7995 100644 --- a/Documentation/technical/multi-pack-index.txt +++ b/Documentation/technical/multi-pack-index.txt @@ -36,8 +36,6 @@ Design Details directory of an alternate. It refers only to packfiles in that same directory. -- The core.multiPackIndex config setting must be on to consume MIDX files. - - The file format includes parameters for the object ID hash function, so a future change of hash algorithm does not require a change in format. ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] doc/technical: remove outdated MIDX default note 2021-09-22 22:13 [PATCH] doc/technical: remove outdated MIDX default note Eric Wong @ 2021-09-23 13:47 ` Derrick Stolee 2021-09-23 17:45 ` Taylor Blau 0 siblings, 1 reply; 9+ messages in thread From: Derrick Stolee @ 2021-09-23 13:47 UTC (permalink / raw) To: Eric Wong, git; +Cc: Derrick Stolee, Junio C Hamano On 9/22/2021 6:13 PM, Eric Wong wrote: > Multi-pack-index files are used by default since > commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the > outdated note was misleading. You are correct that the note is misleading, but it is still correct. A user can _disable_ reading MIDX files by setting core.multiPackIndex to false. If this was in the user-facing docs, not the technical docs, then I might push back on trying to make that distinction. > -- The core.multiPackIndex config setting must be on to consume MIDX files. > - As it is, this deletion is fine. A nit on your commit message: format commit references like this: 18e449f86 (midx: enable core.multiPackIndex by default, 2020-09-25) Thanks, -Stolee ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] doc/technical: remove outdated MIDX default note 2021-09-23 13:47 ` Derrick Stolee @ 2021-09-23 17:45 ` Taylor Blau 2021-09-23 20:12 ` [PATCH v2] doc/technical: update note about core.multiPackIndex Eric Wong 2021-09-23 20:27 ` [PATCH] doc/technical: remove outdated MIDX default note Junio C Hamano 0 siblings, 2 replies; 9+ messages in thread From: Taylor Blau @ 2021-09-23 17:45 UTC (permalink / raw) To: Derrick Stolee; +Cc: Eric Wong, git, Derrick Stolee, Junio C Hamano On Thu, Sep 23, 2021 at 09:47:03AM -0400, Derrick Stolee wrote: > On 9/22/2021 6:13 PM, Eric Wong wrote: > > Multi-pack-index files are used by default since > > commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the > > outdated note was misleading. > > You are correct that the note is misleading, but it is still correct. > > A user can _disable_ reading MIDX files by setting core.multiPackIndex > to false. If this was in the user-facing docs, not the technical docs, > then I might push back on trying to make that distinction. It may present a fuller picture to instead say: The core.multiPackIndex config setting must be on (which is the default) to consume MIDX files. Setting it to `false` prevents Git from reading a MIDX file, even if one exists. Thanks, Taylor ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2] doc/technical: update note about core.multiPackIndex 2021-09-23 17:45 ` Taylor Blau @ 2021-09-23 20:12 ` Eric Wong 2021-09-23 20:21 ` Taylor Blau 2021-09-24 11:11 ` [PATCH v3] " Eric Wong 2021-09-23 20:27 ` [PATCH] doc/technical: remove outdated MIDX default note Junio C Hamano 1 sibling, 2 replies; 9+ messages in thread From: Eric Wong @ 2021-09-23 20:12 UTC (permalink / raw) To: Taylor Blau; +Cc: Derrick Stolee, git, Derrick Stolee, Junio C Hamano Taylor Blau <me@ttaylorr.com> wrote: > On Thu, Sep 23, 2021 at 09:47:03AM -0400, Derrick Stolee wrote: > > On 9/22/2021 6:13 PM, Eric Wong wrote: > > > Multi-pack-index files are used by default since > > > commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the > > > outdated note was misleading. > > > > You are correct that the note is misleading, but it is still correct. > > > > A user can _disable_ reading MIDX files by setting core.multiPackIndex > > to false. If this was in the user-facing docs, not the technical docs, > > then I might push back on trying to make that distinction. > > It may present a fuller picture to instead say: > > The core.multiPackIndex config setting must be on (which is the default) > to consume MIDX files. Setting it to `false` prevents Git from reading a > MIDX file, even if one exists. Ah, thanks, I'll just use your text. ---------8<---------- Subject: [PATCH] doc/technical: update note about core.multiPackIndex MIDX files are used by default since commit d0fa539bc569 (doc/technical: remove outdated MIDX default note, 2021-09-22) Helped-by: Taylor Blau <me@ttaylorr.com> Signed-off-by: Eric Wong <e@80x24.org> --- Documentation/technical/multi-pack-index.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/technical/multi-pack-index.txt b/Documentation/technical/multi-pack-index.txt index 1a73c3ee203e..86f40f24909a 100644 --- a/Documentation/technical/multi-pack-index.txt +++ b/Documentation/technical/multi-pack-index.txt @@ -36,7 +36,9 @@ Design Details directory of an alternate. It refers only to packfiles in that same directory. -- The core.multiPackIndex config setting must be on to consume MIDX files. +- The core.multiPackIndex config setting must be on (which is the + default) to consume MIDX files. Setting it to `false` prevents + Git from reading a MIDX file, even if one exists. - The file format includes parameters for the object ID hash function, so a future change of hash algorithm does not require ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2] doc/technical: update note about core.multiPackIndex 2021-09-23 20:12 ` [PATCH v2] doc/technical: update note about core.multiPackIndex Eric Wong @ 2021-09-23 20:21 ` Taylor Blau 2021-09-24 11:11 ` [PATCH v3] " Eric Wong 1 sibling, 0 replies; 9+ messages in thread From: Taylor Blau @ 2021-09-23 20:21 UTC (permalink / raw) To: Eric Wong Cc: Taylor Blau, Derrick Stolee, git, Derrick Stolee, Junio C Hamano On Thu, Sep 23, 2021 at 08:12:52PM +0000, Eric Wong wrote: > Subject: [PATCH] doc/technical: update note about core.multiPackIndex > > MIDX files are used by default since commit d0fa539bc569 > (doc/technical: remove outdated MIDX default note, 2021-09-22) > > Helped-by: Taylor Blau <me@ttaylorr.com> > Signed-off-by: Eric Wong <e@80x24.org> > --- Thanks; this version looks good to me. Thanks, Taylor ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3] doc/technical: update note about core.multiPackIndex 2021-09-23 20:12 ` [PATCH v2] doc/technical: update note about core.multiPackIndex Eric Wong 2021-09-23 20:21 ` Taylor Blau @ 2021-09-24 11:11 ` Eric Wong 2021-09-25 17:17 ` Derrick Stolee 1 sibling, 1 reply; 9+ messages in thread From: Eric Wong @ 2021-09-24 11:11 UTC (permalink / raw) To: Junio C Hamano; +Cc: Derrick Stolee, git, Derrick Stolee, Taylor Blau MIDX files are used by default since commit 18e449f86b74 (midx: enable core.multiPackIndex by default, 2020-09-25) Helped-by: Taylor Blau <me@ttaylorr.com> Signed-off-by: Eric Wong <e@80x24.org> --- v3: I cited the wrong commit in the commit message of v2 :x Documentation/technical/multi-pack-index.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/technical/multi-pack-index.txt b/Documentation/technical/multi-pack-index.txt index 1a73c3ee203e..86f40f24909a 100644 --- a/Documentation/technical/multi-pack-index.txt +++ b/Documentation/technical/multi-pack-index.txt @@ -36,7 +36,9 @@ Design Details directory of an alternate. It refers only to packfiles in that same directory. -- The core.multiPackIndex config setting must be on to consume MIDX files. +- The core.multiPackIndex config setting must be on (which is the + default) to consume MIDX files. Setting it to `false` prevents + Git from reading a MIDX file, even if one exists. - The file format includes parameters for the object ID hash function, so a future change of hash algorithm does not require ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v3] doc/technical: update note about core.multiPackIndex 2021-09-24 11:11 ` [PATCH v3] " Eric Wong @ 2021-09-25 17:17 ` Derrick Stolee 0 siblings, 0 replies; 9+ messages in thread From: Derrick Stolee @ 2021-09-25 17:17 UTC (permalink / raw) To: Eric Wong, Junio C Hamano; +Cc: git, Derrick Stolee, Taylor Blau On 9/24/2021 7:11 AM, Eric Wong wrote: > MIDX files are used by default since commit 18e449f86b74 > (midx: enable core.multiPackIndex by default, 2020-09-25) > > Helped-by: Taylor Blau <me@ttaylorr.com> > Signed-off-by: Eric Wong <e@80x24.org> > --- > v3: I cited the wrong commit in the commit message of v2 :x Good catch. > Documentation/technical/multi-pack-index.txt | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/technical/multi-pack-index.txt b/Documentation/technical/multi-pack-index.txt > index 1a73c3ee203e..86f40f24909a 100644 > --- a/Documentation/technical/multi-pack-index.txt > +++ b/Documentation/technical/multi-pack-index.txt > @@ -36,7 +36,9 @@ Design Details > directory of an alternate. It refers only to packfiles in that > same directory. > > -- The core.multiPackIndex config setting must be on to consume MIDX files. > +- The core.multiPackIndex config setting must be on (which is the > + default) to consume MIDX files. Setting it to `false` prevents > + Git from reading a MIDX file, even if one exists. > > - The file format includes parameters for the object ID hash > function, so a future change of hash algorithm does not require This looks good to me, too! Thanks, -Stolee ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] doc/technical: remove outdated MIDX default note 2021-09-23 17:45 ` Taylor Blau 2021-09-23 20:12 ` [PATCH v2] doc/technical: update note about core.multiPackIndex Eric Wong @ 2021-09-23 20:27 ` Junio C Hamano 2021-09-23 20:32 ` Taylor Blau 1 sibling, 1 reply; 9+ messages in thread From: Junio C Hamano @ 2021-09-23 20:27 UTC (permalink / raw) To: Taylor Blau; +Cc: Derrick Stolee, Eric Wong, git, Derrick Stolee Taylor Blau <me@ttaylorr.com> writes: > On Thu, Sep 23, 2021 at 09:47:03AM -0400, Derrick Stolee wrote: >> On 9/22/2021 6:13 PM, Eric Wong wrote: >> > Multi-pack-index files are used by default since >> > commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the >> > outdated note was misleading. >> >> You are correct that the note is misleading, but it is still correct. >> >> A user can _disable_ reading MIDX files by setting core.multiPackIndex >> to false. If this was in the user-facing docs, not the technical docs, >> then I might push back on trying to make that distinction. > > It may present a fuller picture to instead say: > > The core.multiPackIndex config setting must be on (which is the default) > to consume MIDX files. Setting it to `false` prevents Git from reading a > MIDX file, even if one exists. Do you want to replace the one-line sentence with that paragraph, instead of removing it, in a replacement patch? ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] doc/technical: remove outdated MIDX default note 2021-09-23 20:27 ` [PATCH] doc/technical: remove outdated MIDX default note Junio C Hamano @ 2021-09-23 20:32 ` Taylor Blau 0 siblings, 0 replies; 9+ messages in thread From: Taylor Blau @ 2021-09-23 20:32 UTC (permalink / raw) To: Junio C Hamano Cc: Taylor Blau, Derrick Stolee, Eric Wong, git, Derrick Stolee On Thu, Sep 23, 2021 at 01:27:31PM -0700, Junio C Hamano wrote: > Taylor Blau <me@ttaylorr.com> writes: > > > On Thu, Sep 23, 2021 at 09:47:03AM -0400, Derrick Stolee wrote: > >> On 9/22/2021 6:13 PM, Eric Wong wrote: > >> > Multi-pack-index files are used by default since > >> > commit 18e449f86b74bab35b150549c8342d252fe7ae00, so the > >> > outdated note was misleading. > >> > >> You are correct that the note is misleading, but it is still correct. > >> > >> A user can _disable_ reading MIDX files by setting core.multiPackIndex > >> to false. If this was in the user-facing docs, not the technical docs, > >> then I might push back on trying to make that distinction. > > > > It may present a fuller picture to instead say: > > > > The core.multiPackIndex config setting must be on (which is the default) > > to consume MIDX files. Setting it to `false` prevents Git from reading a > > MIDX file, even if one exists. > > Do you want to replace the one-line sentence with that paragraph, > instead of removing it, in a replacement patch? That's what I was hoping for, and Eric did that here: https://lore.kernel.org/git/20210923201252.GA25980@dcvr/ Thanks, Taylor ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-09-25 17:18 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-09-22 22:13 [PATCH] doc/technical: remove outdated MIDX default note Eric Wong 2021-09-23 13:47 ` Derrick Stolee 2021-09-23 17:45 ` Taylor Blau 2021-09-23 20:12 ` [PATCH v2] doc/technical: update note about core.multiPackIndex Eric Wong 2021-09-23 20:21 ` Taylor Blau 2021-09-24 11:11 ` [PATCH v3] " Eric Wong 2021-09-25 17:17 ` Derrick Stolee 2021-09-23 20:27 ` [PATCH] doc/technical: remove outdated MIDX default note Junio C Hamano 2021-09-23 20:32 ` Taylor Blau
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.