threads / patch / 3119

patchAsciiDoc fixes for the git-svnimport manpage

Subject: [PATCH] AsciiDoc fixes for the git-svnimport manpage

## tl;dr

3 messages between Jan 21, 2006 and Jan 21, 2006. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Florian Weimer· Jan 21, 2006, 19:06 UTC · lore

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

Show changes to Documentation/git-svnimport.txt +4 −8

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

Junio C Hamano· Jan 21, 2006, 19:24 UTC · re: Florian Weimer · lore

Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage

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"?

Show 7 quoted lines

> -<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...

Florian Weimer· Jan 21, 2006, 19:31 UTC · re: Junio C Hamano · lore

Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage

* Junio C. Hamano:

Show 6 quoted lines

> 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.

Show 11 quoted lines

>> -<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://).