From: Junio C Hamano <junkio@cox.net>
To: Eric Wong <normalperson@yhbt.net>
Cc: git@vger.kernel.org
Subject: minimize_url in git-svn?
Date: Tue, 08 May 2007 02:58:32 -0700 [thread overview]
Message-ID: <7v4pmn4oxj.fsf_-_@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <464023D4.5060101@midwinter.com> (Steven Grimm's message of "Tue, 08 May 2007 00:16:36 -0700")
I was trying to run git-svn against this:
https://repo.socialtext.net:8999/svn/socialtext/trunk
This is an open source project [*1*] and the trunk is supposed
to be readable by everybody, but it seems that anything outside
that area needs authentication. If I mimick the example in
git-svn.txt manual page to clone from there, it creates trunk,
trunk/.git, and then asks for password:
$ URL=https://repo.socialtext.net:8999/svn/socialtext/trunk
$ git-svn clone $URL
Authentication realm: <https://repo.socialtext.net:8999> Auth for SVN
Password for 'junio': ^C
I've narrowed it down to this part of git-svn. If I tell it not
to bother "minimiz"ing the URL, it seems to import without
stepping outside of the URL it was given.
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -1038,7 +1038,8 @@
}
$self->{repo_id} = $existing;
} else {
- my $min_url = Git::SVN::Ra->new($url)->minimize_url;
+ my $ra = Git::SVN::Ra->new($url);
+ my $min_url = $url; # $ra->minimize_url;
$existing = find_existing_remote($min_url, $r);
if ($existing) {
unless ($no_write) {
Two and half questions.
* What does minimize do, and why is it necessary?
* The resulting git-svn remote tracking branch (and 'master')
seems to check out fine, but I do not know what damage the
hack to avoid minimizing is causing. Are there any? I see
many 0{40} lines in trunk/.git/svn/git-svn/.rev_db.* file,
and also many lines in unhandled.log file (+empty_dir,
+file_prop, and +dir_prop). Are these something to worry
about?
* Assuming there aren't any damage, or maybe some damage that
would cause minor decreased functionality/interoperability,
would it perhaps make sense to optionally allow skipping the
minimizing to avoid this problem? Would it make sense, or is
the setting at socialtext site too esoteric and it isn't
worth to worry about?
[Footnote]
*1* http://www.socialtext.net/stoss/index.cgi?developing_with_a_dev_env
next prev parent reply other threads:[~2007-05-08 9:58 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-06 18:56 importing multi-project svn repositories David Hanson
2007-05-07 18:06 ` Steven Grimm
2007-05-08 3:48 ` Dave Hanson
2007-05-08 7:16 ` Deprecate git-svnimport? Steven Grimm
2007-05-08 9:58 ` Junio C Hamano [this message]
2007-05-08 11:28 ` minimize_url in git-svn? Johannes Schindelin
2007-05-08 19:34 ` Eric Wong
2007-05-09 4:56 ` Junio C Hamano
2007-05-13 16:58 ` [PATCH] git-svn: don't attempt to minimize URLs by default Eric Wong
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=7v4pmn4oxj.fsf_-_@assigned-by-dhcp.cox.net \
--to=junkio@cox.net \
--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).