Re: [PATCH] git-pack-objects.txt: fix grammatical errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 2 Apr 2011, Junio C Hamano wrote:

> Nicolas Pitre <nico@xxxxxxxxxxx> writes:
> 
> >> diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
> >> index a51071e..442018b 100644
> >> --- a/Documentation/git-pack-objects.txt
> >> +++ b/Documentation/git-pack-objects.txt
> >> @@ -190,9 +190,9 @@ self-contained. Use `git index-pack --fix-thin`
> >>  (see linkgit:git-index-pack[1]) to restore the self-contained property.
> >>  
> >>  --delta-base-offset::
> >> +	A packed archive can express the base object of a delta as
> >> +	either a 20-byte object name or as an offset in the
> >> +	stream, but older versions of git don't understand the
> >>  	latter.  By default, 'git pack-objects' only uses the
> >>  	former format for better compatibility.  This option
> >>  	allows the command to use the latter format for
> >
> > This is misleading and not even true anymore if not using pack-objects.  
> > Most people are using either 'git repack' or even 'git gc', and the 
> > default is to use this parameter since commit 22c79eab.
> 
> This page is on the pack-objects plumbing, and in that context there is
> nothing misleading nor false about it.

True.  But...

> We may however want to do something like the attached to help people who
> might wonder how to pass this option through the Porcelain they use.

That's the kind of precision I was meaning.

> the longer term, it probably makes sense to make the option default at a
> version bump, perhaps at the 1.8.0 boundary.

Agreed.


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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]