mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
![Eric Wong](/assets/img/avatar_default.png)
And then unescape them when writing to $GIT_CONFIG.
SVN has different rules for repository URLs (usually the root)
and for paths within that repository (below the HTTP layer).
Thus, for the request URI path at the HTTP level, the URI needs
to be encoded. However, in the body of the HTTP request (the
with underlying SVN XML protocol), those paths should not be
URI-encoded[1]. For non-HTTP(S) requests, SVN appears to be
more flexible and will except weird characters in the URL as
well as URI-encoded ones.
Since users are used to using URLs being entirely URI-encoded,
git svn will now attempt to unescape the path portion of URLs
while leaving the actual repository URL untouched.
This change will be reflected in newly-created $GIT_CONFIG files
only. This allows users to switch between svn(+ssh)://, file://
and http(s):// urls without changing the fetch/branches/tags
config keys. This won't affect existing imports at all (since
things didn't work before this commit anyways), and will allow
users to force escaping into repository paths that look like
they're escaped (but are not).
Thanks to Mike Smullin for the original bug report and Björn
Steinbrink for summarizing it into testable cases for me.
[1] Except when committing copies/renames, see
commit 29633bb91c
Signed-off-by: Eric Wong <normalperson@yhbt.net>
79 lines
2 KiB
Bash
Executable file
79 lines
2 KiB
Bash
Executable file
#!/bin/sh
|
|
#
|
|
# Copyright (c) 2008 Kevin Ballard
|
|
#
|
|
|
|
test_description='git svn clone with percent escapes'
|
|
. ./lib-git-svn.sh
|
|
|
|
test_expect_success 'setup svnrepo' '
|
|
mkdir project project/trunk project/branches project/tags &&
|
|
echo foo > project/trunk/foo &&
|
|
svn_cmd import -m "$test_description" project "$svnrepo/pr ject" &&
|
|
svn_cmd cp -m "branch" "$svnrepo/pr ject/trunk" \
|
|
"$svnrepo/pr ject/branches/b" &&
|
|
svn_cmd cp -m "tag" "$svnrepo/pr ject/trunk" \
|
|
"$svnrepo/pr ject/tags/v1" &&
|
|
rm -rf project &&
|
|
start_httpd
|
|
'
|
|
|
|
test_expect_success 'test clone with percent escapes' '
|
|
git svn clone "$svnrepo/pr%20ject" clone &&
|
|
cd clone &&
|
|
git rev-parse refs/${remotes_git_svn} &&
|
|
cd ..
|
|
'
|
|
|
|
# SVN works either way, so should we...
|
|
|
|
test_expect_success 'svn checkout with percent escapes' '
|
|
svn_cmd checkout "$svnrepo/pr%20ject" svn.percent &&
|
|
svn_cmd checkout "$svnrepo/pr%20ject/trunk" svn.percent.trunk
|
|
'
|
|
|
|
test_expect_success 'svn checkout with space' '
|
|
svn_cmd checkout "$svnrepo/pr ject" svn.space &&
|
|
svn_cmd checkout "$svnrepo/pr ject/trunk" svn.space.trunk
|
|
'
|
|
|
|
test_expect_success 'test clone trunk with percent escapes and minimize-url' '
|
|
git svn clone --minimize-url "$svnrepo/pr%20ject/trunk" minimize &&
|
|
(
|
|
cd minimize &&
|
|
git rev-parse refs/${remotes_git_svn}
|
|
)
|
|
'
|
|
|
|
test_expect_success 'test clone trunk with percent escapes' '
|
|
git svn clone "$svnrepo/pr%20ject/trunk" trunk &&
|
|
(
|
|
cd trunk &&
|
|
git rev-parse refs/${remotes_git_svn}
|
|
)
|
|
'
|
|
|
|
test_expect_success 'test clone --stdlayout with percent escapes' '
|
|
git svn clone --stdlayout "$svnrepo/pr%20ject" percent &&
|
|
(
|
|
cd percent &&
|
|
git rev-parse refs/remotes/trunk^0 &&
|
|
git rev-parse refs/remotes/b^0 &&
|
|
git rev-parse refs/remotes/tags/v1^0
|
|
)
|
|
'
|
|
|
|
test_expect_success 'test clone -s with unescaped space' '
|
|
git svn clone -s "$svnrepo/pr ject" space &&
|
|
(
|
|
cd space &&
|
|
git rev-parse refs/remotes/trunk^0 &&
|
|
git rev-parse refs/remotes/b^0 &&
|
|
git rev-parse refs/remotes/tags/v1^0
|
|
)
|
|
'
|
|
|
|
stop_httpd
|
|
|
|
test_done
|