[PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[Petr Baudis]

Note that v1.4.4.5 supports pack index v2, and describe how to keep
your repositories backwards-compatible, shall you need to.

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 Documentation/RelNotes-1.6.0.txt |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/Documentation/RelNotes-1.6.0.txt b/Documentation/RelNotes-1.6.0.txt
index b29ba25..9bbb07e 100644
--- a/Documentation/RelNotes-1.6.0.txt
+++ b/Documentation/RelNotes-1.6.0.txt
@@ -21,7 +21,9 @@ main git.git codebase.
 By default, packfiles created with this version uses delta-base-offset
 encoding introduced in v1.4.4.  Pack idx files are using version 2 that
 allows larger packs and added robustness thanks to its CRC checking,
-introduced in v1.5.2.
+introduced in v1.5.2 and v1.4.4.5.  If you want to keep your repositories
+backwards compatible past these versions, set repack.useDeltaBaseOffset
+to false or pack.indexVersion to 1, respectively.
 
 GIT_CONFIG, which was only documented as affecting "git config", but
 actually affected all git commands, now only affects "git config".

Re: [PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[Jakub Narebski]

Petr Baudis wrote:

> +introduced in v1.5.2 and v1.4.4.5.  If you want to keep your repositories
> +backwards compatible past these versions, set repack.useDeltaBaseOffset
> +to false or pack.indexVersion to 1, respectively.

'or'? Not 'and'?

You shouldn't have, I think, this "respectively" here.  You need it only
if you are writing "a, b, c, then a', b', c', respectively".

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

Re: [PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[Petr Baudis]

On Thu, Jul 17, 2008 at 07:10:32PM +0200, Jakub Narebski wrote:
> Petr Baudis wrote:
> 
> > +introduced in v1.5.2 and v1.4.4.5.  If you want to keep your repositories
> > +backwards compatible past these versions, set repack.useDeltaBaseOffset
> > +to false or pack.indexVersion to 1, respectively.
> 
> 'or'? Not 'and'?
> 
> You shouldn't have, I think, this "respectively" here.  You need it only
> if you are writing "a, b, c, then a', b', c', respectively".

The "respectively" means that the two options relate each to one of the
two new features. I'm unclear about the 'or'-'and' question, though.

				Petr "Pasky" Baudis

Re: [PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[Jakub Narebski]

Petr Baudis wrote:
> On Thu, Jul 17, 2008 at 07:10:32PM +0200, Jakub Narebski wrote:
> > Petr Baudis wrote:
> > 
> > > +introduced in v1.5.2 and v1.4.4.5.  If you want to keep your repositories
> > > +backwards compatible past these versions, set repack.useDeltaBaseOffset
> > > +to false or pack.indexVersion to 1, respectively.
> > 
> > 'or'? Not 'and'?
> > 
> > You shouldn't have, I think, this "respectively" here.  You need it only
> > if you are writing "a, b, c, then a', b', c', respectively".
> 
> The "respectively" means that the two options relate each to one of the
> two new features.

Sorry, I have forgot that "respectively" might refer to the context
which I don't see in a patch.

> I'm unclear about the 'or'-'and' question, though. 

If you want to be backwards compatibile with pre v1.4.4.5 clients,
do you have to set either option, or both options, or what?  The
same with pre v1.5.2 client...

-- 
Jakub Narebski
Poland

Re: [PATCH] Documentation/RelNotes-1.6.0.txt: Expand on the incompatible packfiles

[Petr Baudis]

On Thu, Jul 17, 2008 at 07:40:06PM +0200, Jakub Narebski wrote:
> Petr Baudis wrote:
> > I'm unclear about the 'or'-'and' question, though. 
> 
> If you want to be backwards compatibile with pre v1.4.4.5 clients,
> do you have to set either option, or both options, or what?  The
> same with pre v1.5.2 client...

If you want to be backwards compatible with _either_ pre-v1.4.4 or
pre-v1.5.2, you will set _either_ (a) or (b). Setting (a) implies you
want to set (b) too, though. :-)

				Petr "Pasky" Baudis