From: Florian Weimer <fw@deneb.enyo.de>
Date: Sat, 21 Jan 2006 19:31:09 GMT
Subject: Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage
Message-ID: <8764od8jk2.fsf@mid.deneb.enyo.de>
In-Reply-To: <7vslrho038.fsf@assigned-by-dhcp.cox.net>

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