public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
* [PATCH] AsciiDoc fixes for the git-svnimport manpage
@ 2006-01-21 19:06 Florian Weimer
  2006-01-21 19:24 ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Florian Weimer @ 2006-01-21 19:06 UTC (permalink / raw)


Change "SVN:: Perl" to "SVN::Perl", wrap a long line, and clean up the
description of positional arguments.

Signed-off-by: Florian Weimer <fw@deneb•enyo.de>

---

 Documentation/git-svnimport.txt |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

592699f759a61aee33060d91cfe13ff432c147cf
diff --git a/Documentation/git-svnimport.txt b/Documentation/git-svnimport.txt
index db1ce38..63e28b8 100644
--- a/Documentation/git-svnimport.txt
+++ b/Documentation/git-svnimport.txt
@@ -21,7 +21,7 @@ DESCRIPTION
 Imports a SVN repository into git. It will either create a new
 repository, or incrementally import into an existing one.
 
-SVN access is done by the SVN:: Perl module.
+SVN access is done by the SVN::Perl module.
 
 git-svnimport assumes that SVN repositories are organized into one
 "trunk" directory where the main development happens, "branch/FOO"
@@ -74,8 +74,8 @@ When importing incrementally, you might 
 -l <max_rev>::
 	Specify a maximum revision number to pull.
 
-	Formerly, this option controlled how many revisions to pull, due to
-	SVN memory leaks. (These have been worked around.)
+	Formerly, this option controlled how many revisions to pull,
+	due to SVN memory leaks. (These have been worked around.)
 
 -v::
 	Verbosity: let 'svnimport' report what it is doing.
@@ -100,11 +100,7 @@ with a 40x error pretty quickly.
 If you're using the "-d" or "-D" option, this is the URL of the SVN
 repository itself; it usually ends in "/svn".
 
-<SVN_repository_URL>::
-	The URL of the SVN module you want to import. For local
-	repositories, use "file:///absolute/path".
-
-<path>
+<path>::
 	The path to the module you want to check out.
 
 -h::
-- 
1.1.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage
  2006-01-21 19:06 [PATCH] AsciiDoc fixes for the git-svnimport manpage Florian Weimer
@ 2006-01-21 19:24 ` Junio C Hamano
  2006-01-21 19:31   ` Florian Weimer
  0 siblings, 1 reply; 3+ messages in thread
From: Junio C Hamano @ 2006-01-21 19:24 UTC (permalink / raw)
  To: Florian Weimer; +Cc: git

Florian Weimer <fw@deneb•enyo.de> writes:

> Change "SVN:: Perl" to "SVN::Perl", wrap a long line, and clean up the
> description of positional arguments.

Thanks.  Is this the part you mean by "positional arguments"?

> -<SVN_repository_URL>::
> -	The URL of the SVN module you want to import. For local
> -	repositories, use "file:///absolute/path".
> -
> -<path>
> +<path>::
>  	The path to the module you want to check out.

This looks to me as if we do not allow non file:/// URL like
svn:// anymore, or we never supported it but the documentation
pretended we did.  Is that what you intended to fix?

I am a bit confused...

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage
  2006-01-21 19:24 ` Junio C Hamano
@ 2006-01-21 19:31   ` Florian Weimer
  0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2006-01-21 19:31 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

* Junio C. Hamano:

> Florian Weimer <fw@deneb•enyo.de> writes:
>
>> Change "SVN:: Perl" to "SVN::Perl", wrap a long line, and clean up the
>> description of positional arguments.
>
> Thanks.  Is this the part you mean by "positional arguments"?

Uhm, "positional arguments" are non-option arguments.

>> -<SVN_repository_URL>::
>> -	The URL of the SVN module you want to import. For local
>> -	repositories, use "file:///absolute/path".
>> -
>> -<path>
>> +<path>::
>>  	The path to the module you want to check out.
>
> This looks to me as if we do not allow non file:/// URL like
> svn:// anymore, or we never supported it but the documentation
> pretended we did.  Is that what you intended to fix?

No, there was a :: missing, and the SVN_repository_URL part was
duplicated.  The documentation tries to remind the reader that the
"/absolute/path" syntax does not work and you have to use a real URL.

(I haven't got git-svnimport to work so far, but the Subversion RA
layer is transport-agnostic, so if it works for file://, it will also
work for svn:// or http://).

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-01-21 19:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-01-21 19:06 [PATCH] AsciiDoc fixes for the git-svnimport manpage Florian Weimer
2006-01-21 19:24 ` Junio C Hamano
2006-01-21 19:31   ` Florian Weimer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox