public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: Florian Weimer <fw@deneb•enyo.de>
To: Junio C Hamano <junkio@cox•net>
Cc: git@vger•kernel.org
Subject: Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage
Date: Sat, 21 Jan 2006 20:31:09 +0100	[thread overview]
Message-ID: <8764od8jk2.fsf@mid.deneb.enyo.de> (raw)
In-Reply-To: <7vslrho038.fsf@assigned-by-dhcp.cox.net> (Junio C. Hamano's message of "Sat, 21 Jan 2006 11:24:59 -0800")

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

      reply	other threads:[~2006-01-21 19:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8764od8jk2.fsf@mid.deneb.enyo.de \
    --to=fw@deneb$(echo .)enyo.de \
    --cc=git@vger$(echo .)kernel.org \
    --cc=junkio@cox$(echo .)net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox