git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: Eric Wong <normalperson@yhbt.net>
Cc: git <git@vger.kernel.org>
Subject: SVN_URL undocumented
Date: Mon, 10 Apr 2006 17:22:42 -0400	[thread overview]
Message-ID: <1144704162.18727.30.camel@dv> (raw)

Hello, Eric!

I'm trying to use git-svn, but the documentation seems to be quite
unclear.  I'm using current git-svn from the git repository.

I'm running "git-svn init" in a Subversion working directory:

$ git-svn init
SVN repository location required
 at /home/proski/bin/git-svn line 223
        main::init() called at /home/proski/bin/git-svn line 104

The help shown by git-svn without arguments doesn't say anything about
arguments for "init", unlike other subcommands.  I'm assuming that
"git-svn init" is not taking any arguments.

The manual page doesn't say anything about arguments, but it mentions
that "the SVN_URL must be specified at this point".  This statement is
unclear to me.

There are several references to SVN_URL in the manual, but there is no
description what it is.  Is it supposed to be an environment variable,
or an undocumented argument, or is it taken from "svn info" output?

The environment variable has no effect:

$ SVN_URL=http://svn.madwifi.org/trunk git-svn init
SVN repository location required
 at /home/proski/bin/git-svn line 223
        main::init() called at /home/proski/bin/git-svn line 104

Looking at the sources, it looks like that SVN_URL is an undocumented
argument for "git-cvs init".  "git-svn fetch" appears to be taking that
value from the git repository, but it also takes care to protect the
original value (I have no idea where it can come from).

One of the "basic examples" further in the manual confirms that SVN_URL
is set on the command line:

git-svn init http://svn.foo.org/project/trunk

I believe that the documentation should be updated to describe where
SVN_URL comes from, and what arguments are accepted by "git-svn init".
Basic examples shouldn't recommend using undocumented arguments.

It seems to me that the current documentation describes the internals of
git-svn instead of its user interface.  Rather than tell the user that
"FOO must be specified at that point", the documentation should say what
(an how) should (or can) be specified when the command is invoked on the
command line.

Also, it would be great to refer to the "additional fetch arguments" in
the "fetch" description.  That section is easy to miss by somebody who
is specifically looking for "fetch arguments".

-- 
Regards,
Pavel Roskin

                 reply	other threads:[~2006-04-10 21:22 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=1144704162.18727.30.camel@dv \
    --to=proski@gnu.org \
    --cc=git@vger.kernel.org \
    --cc=normalperson@yhbt.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).