threads / patch / 32534

patchgit-fast-import(1): remove duplicate "--done" option

Subject: [PATCH] git-fast-import(1): remove duplicate "--done" option

## tl;dr

5 messages between Jan 5, 2013 and Jan 8, 2013. Diffs are folded; open one to read it.

replies: 4people: 3as markdown or json

John Keeping· Jan 5, 2013, 16:06 UTC · lore

The "--done" option to git-fast-import is documented twice in its manual page. Combine the best bits of each description, keeping the location of the instance that was added first.

Signed-off-by: John Keeping <john@keeping.me.uk>
---
I'm guessing that the reason the option was documented again (in commit
3266de10) is because the options are not in an obvious order.  There
does seem to be some grouping of the options by type, but without
subheadings I wonder if it would make more sense to just put them all in
alphabetical order?
 Documentation/git-fast-import.txt | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
Show changes to Documentation/git-fast-import.txt +2 −4
diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index 68bca1a..4ef5721 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -39,10 +39,6 @@ OPTIONS
 	See ``Date Formats'' below for details about which formats
 	are supported, and their syntax.
 
--- done::
-	Terminate with error if there is no 'done' command at the
-	end of the stream.
-
 --force::
 	Force updating modified existing branches, even if doing
 	so would cause commits to be lost (as the new commit does
@@ -108,7 +104,8 @@ OPTIONS
 	output.
 
 --done::
-	Require a `done` command at the end of the stream.
+	Terminate with error if there is no 'done' command at the
+	end of the stream.
 	This option might be useful for detecting errors that
 	cause the frontend to terminate before it has started to
 	write a stream.
-- 
1.8.0.2
Eric S. Raymond· Jan 5, 2013, 16:26 UTC · re: John Keeping · lore

Re: [PATCH] git-fast-import(1): remove duplicate "--done" option

John Keeping <john@keeping.me.uk>:
Show 5 quoted lines
> I'm guessing that the reason the option was documented again (in commit
> 3266de10) is because the options are not in an obvious order.  There
> does seem to be some grouping of the options by type, but without
> subheadings I wonder if it would make more sense to just put them all in
> alphabetical order?
+1

This duplication originated with me. I'll apologize with a reordering patch.

-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>
Jonathan Nieder· Jan 5, 2013, 23:12 UTC · re: John Keeping · lore

Re: [PATCH] git-fast-import(1): remove duplicate "--done" option

John Keeping wrote:
Show 5 quoted lines
> The "--done" option to git-fast-import is documented twice in its manual
> page.  Combine the best bits of each description, keeping the location
> of the instance that was added first.
>
> Signed-off-by: John Keeping <john@keeping.me.uk>
Good catch, thanks.
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
John Keeping· Jan 7, 2013, 11:57 UTC · re: John Keeping · lore

[PATCH v2] git-fast-import(1): remove duplicate '--done' option

On Sat, Jan 05, 2013 at 04:06:52PM +0000, John Keeping wrote: The '--done' option to git-fast-import is documented twice in its manual page. Combine the best bits of each description, keeping the location of the instance that was added first.

Signed-off-by: John Keeping <john@keeping.me.uk>
---
Changed since v1:
    'done' => `done`

I'll wait a bit longer for comments on [1] before I have another go at the full re-organisation of this page.

[1] http://article.gmane.org/gmane.comp.version-control.git/212805
 Documentation/git-fast-import.txt | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
Show changes to Documentation/git-fast-import.txt +2 −4
diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index 68bca1a..4ef5721 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -39,10 +39,6 @@ OPTIONS
 	See ``Date Formats'' below for details about which formats
 	are supported, and their syntax.
 
--- done::
-	Terminate with error if there is no 'done' command at the
-	end of the stream.
-
 --force::
 	Force updating modified existing branches, even if doing
 	so would cause commits to be lost (as the new commit does
@@ -108,7 +104,8 @@ OPTIONS
 	output.
 
 --done::
-	Require a `done` command at the end of the stream.
+	Terminate with error if there is no `done` command at the
+	end of the stream.
 	This option might be useful for detecting errors that
 	cause the frontend to terminate before it has started to
 	write a stream.
-- 
1.8.0.2
John Keeping· Jan 8, 2013, 19:39 UTC · re: John Keeping · lore

[PATCH v3] git-fast-import(1): remove duplicate '--done' option

The '--done' option to git-fast-import is documented twice in its manual page. Combine the best bits of each description, keeping the location of the instance that was added first.

Signed-off-by: John Keeping <john@keeping.me.uk>
---
The commit description gained some noise in v2; this version should be
the really correct, final version.
Sorry for the noise.
 Documentation/git-fast-import.txt | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
Show changes to Documentation/git-fast-import.txt +2 −4
diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index 68bca1a..4ef5721 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -39,10 +39,6 @@ OPTIONS
 	See ``Date Formats'' below for details about which formats
 	are supported, and their syntax.
 
--- done::
-	Terminate with error if there is no 'done' command at the
-	end of the stream.
-
 --force::
 	Force updating modified existing branches, even if doing
 	so would cause commits to be lost (as the new commit does
@@ -108,7 +104,8 @@ OPTIONS
 	output.
 
 --done::
-	Require a `done` command at the end of the stream.
+	Terminate with error if there is no `done` command at the
+	end of the stream.
 	This option might be useful for detecting errors that
 	cause the frontend to terminate before it has started to
 	write a stream.
-- 
1.8.0.2

← back to recent threads