[PATCH resend] Documentation: pack-objects: Clarify --local's semantics.

Nelson Elhage· Feb 19, 2010, 03:29 UTC · lore

The current documentation suggests that --local also ignores any objects in local packs, which is incorrect. Change the language to be clearer and more parallel to the other options that ignore objects.

While we're at it, fix a trivial error in --incremental's documentation.

Signed-off-by: Nelson Elhage <nelhage@mit.edu>
---
 This is an unmodified resend of a patch I sent about a month ago
 which received no response.

 Documentation/git-pack-objects.txt |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

Show changes to Documentation/git-pack-objects.txt +4 −5

diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
index f54d433..d8e5686 100644
--- a/Documentation/git-pack-objects.txt
+++ b/Documentation/git-pack-objects.txt
@@ -117,14 +117,13 @@ base-name::
 	standard input.
 
 --incremental::
-	This flag causes an object already in a pack ignored
+	This flag causes an object already in a pack to be ignored
 	even if it appears in the standard input.
 
 --local::
-	This flag is similar to `--incremental`; instead of
-	ignoring all packed objects, it only ignores objects
-	that are packed and/or not in the local object store
-	(i.e. borrowed from an alternate).
+	This flag causes an object that is borrowed from an alternate
+	object store to be ignored even if it appears in the standard
+	input.
 
 --non-empty::
         Only create a packed archive if it would contain at

-- 
1.6.6.30.g1e6fd

Junio C Hamano· Feb 19, 2010, 05:44 UTC · re: Nelson Elhage · lore

Re: [PATCH resend] Documentation: pack-objects: Clarify --local's semantics.

Nelson Elhage <nelhage@MIT.EDU> writes:

Show 14 quoted lines

> diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
> index f54d433..d8e5686 100644
> --- a/Documentation/git-pack-objects.txt
> +++ b/Documentation/git-pack-objects.txt
> @@ -117,14 +117,13 @@ base-name::
> ...
>  --local::
> -	This flag is similar to `--incremental`; instead of
> -	ignoring all packed objects, it only ignores objects
> -	that are packed and/or not in the local object store
> -	(i.e. borrowed from an alternate).
> +	This flag causes an object that is borrowed from an alternate
> +	object store to be ignored even if it appears in the standard
> +	input.

Thanks.  Comments from "packing" experts?

patchDocumentation: pack-objects: Clarify --local's semantics.