# t9155 fails since "git-svn: fix fetch with deleted tag" with svn < 1.5

7 messages from 2010-08-19 to 2010-08-23. Participants: Jens Lehmann, Ævar Arnfjörð Bjarmason, Eric Wong, David D. Kilzer.
Thread: https://gitlist.dev/t/24793

## Jens Lehmann, 2010-08-19 16:57

Subject: t9155 fails since "git-svn: fix fetch with deleted tag" with svn < 1.5
Message-ID: <4C6D6274.90009@web.de>
URL: https://gitlist.dev/e/4C6D6274.90009%40web.de

```
t9155 fails on current next when running the testsuite on a Fedora
Core 7 system. The reason is that FC7 comes with svn 1.4.4, while
in commit 3235b70 the --parents option was introduced into t9155
which was added in svn 1.5.

What minimum svn version is required for git svn? I didn't find any
information about that, but I might have overlooked something ...

```

## Ævar Arnfjörð Bjarmason, 2010-08-19 17:21

Subject: Re: t9155 fails since "git-svn: fix fetch with deleted tag" with svn < 1.5
Message-ID: <AANLkTiky8qn_PSgXSn=0h07cb4VmCFiFXmD9WrpO1z6z@mail.gmail.com>
URL: https://gitlist.dev/e/AANLkTiky8qn_PSgXSn%3D0h07cb4VmCFiFXmD9WrpO1z6z%40mail.gmail.com
In-Reply-To: <4C6D6274.90009@web.de>

```
On Thu, Aug 19, 2010 at 16:57, Jens Lehmann <Jens.Lehmann@web.de> wrote:
> t9155 fails on current next when running the testsuite on a Fedora
> Core 7 system. The reason is that FC7 comes with svn 1.4.4, while
> in commit 3235b70 the --parents option was introduced into t9155
> which was added in svn 1.5.
>
> What minimum svn version is required for git svn? I didn't find any
> information about that, but I might have overlooked something ...

We currently require >= 1.1.0 of SVN::Core for the git-svn tests at least.

```

## Eric Wong, 2010-08-19 19:14

Subject: Re: t9155 fails since "git-svn: fix fetch with deleted tag" with svn < 1.5
Message-ID: <20100819191425.GA1543@dcvr.yhbt.net>
URL: https://gitlist.dev/e/20100819191425.GA1543%40dcvr.yhbt.net
In-Reply-To: <AANLkTiky8qn_PSgXSn=0h07cb4VmCFiFXmD9WrpO1z6z@mail.gmail.com>

```
Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:
> On Thu, Aug 19, 2010 at 16:57, Jens Lehmann <Jens.Lehmann@web.de> wrote:
> > t9155 fails on current next when running the testsuite on a Fedora
> > Core 7 system. The reason is that FC7 comes with svn 1.4.4, while
> > in commit 3235b70 the --parents option was introduced into t9155
> > which was added in svn 1.5.

The --parents option appears completely unnecessary in this case,
so removing it should be safe.  Patch coming...

> > What minimum svn version is required for git svn? I didn't find any
> > information about that, but I might have overlooked something ...
> 
> We currently require >= 1.1.0 of SVN::Core for the git-svn tests at least.

That's still the only hard requirement I know of.  While I haven't had a
1.1.x installation in years, reports of incompatibility have been few.
So either people find it easier to upgrade to newer versions of SVN,
don't care, or there just aren't that many incompatibilities.

I'd say we will continue to do our best to support whatever versions
people care to report (preferably with patches :) incompatibilities for.

-- 
Eric Wong

```

## Eric Wong, 2010-08-19 19:15

Subject: [PATCH] t9155: fix compatibility with older SVN
Message-ID: <20100819191552.GB1543@dcvr.yhbt.net>
URL: https://gitlist.dev/e/20100819191552.GB1543%40dcvr.yhbt.net
In-Reply-To: <20100819191425.GA1543@dcvr.yhbt.net>

```
The "--parents" option did not appear until SVN 1.5.x
and is completely unnecessary in this case.

Reported-by: Jens Lehmann <Jens.Lehmann@web.de>
Signed-off-by: Eric Wong <normalperson@yhbt.net>
---

 Also pushed up to git://git.bogomips.org/git-svn

 t/t9155-git-svn-fetch-deleted-tag.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/t/t9155-git-svn-fetch-deleted-tag.sh b/t/t9155-git-svn-fetch-deleted-tag.sh
index ef0ac87..a486a98 100755
--- a/t/t9155-git-svn-fetch-deleted-tag.sh
+++ b/t/t9155-git-svn-fetch-deleted-tag.sh
@@ -12,7 +12,7 @@ test_expect_success 'setup svn repo' '
 	svn_cmd import -m "import for git svn" import "$svnrepo" &&
 	rm -rf import &&
 
-	svn_cmd mkdir --parents -m "create mybranch directory" "$svnrepo/branches/mybranch" &&
+	svn_cmd mkdir -m "create mybranch directory" "$svnrepo/branches/mybranch" &&
 	svn_cmd cp -m "create branch mybranch" "$svnrepo/trunk" "$svnrepo/branches/mybranch/trunk" &&
 
 	svn_cmd co "$svnrepo/trunk" svn_project &&
-- 
Eric Wong

```

## David D. Kilzer, 2010-08-19 19:39

Subject: Re: [PATCH] t9155: fix compatibility with older SVN
Message-ID: <56635.12651.qm@web30008.mail.mud.yahoo.com>
URL: https://gitlist.dev/e/56635.12651.qm%40web30008.mail.mud.yahoo.com
In-Reply-To: <20100819191552.GB1543@dcvr.yhbt.net>

```
On Thu, August 19, 2010 at 12:15:52 PM, Eric Wong wrote:


> The "--parents" option did not appear until SVN 1.5.x
> and is completely  unnecessary in this case.


At one point in development, I wasn't creating the branches directory on the 
initial import, which is why this was added.  Thanks for fixing this!

> Reported-by: Jens Lehmann <Jens.Lehmann@web.de>
> Signed-off-by:  Eric Wong <normalperson@yhbt.net>


Ack.

Dave

```

## Jens Lehmann, 2010-08-19 19:40

Subject: Re: [PATCH] t9155: fix compatibility with older SVN
Message-ID: <4C6D8898.1040907@web.de>
URL: https://gitlist.dev/e/4C6D8898.1040907%40web.de
In-Reply-To: <20100819191552.GB1543@dcvr.yhbt.net>

```
Am 19.08.2010 21:15, schrieb Eric Wong:
> The "--parents" option did not appear until SVN 1.5.x
> and is completely unnecessary in this case.

Thanks for the quick response! I'll test this patch on Monday when
I have access to that FC7 machine again.

```

## Jens Lehmann, 2010-08-23 16:22

Subject: Re: [PATCH] t9155: fix compatibility with older SVN
Message-ID: <4C72A063.3010003@web.de>
URL: https://gitlist.dev/e/4C72A063.3010003%40web.de
In-Reply-To: <4C6D8898.1040907@web.de>

```
Am 19.08.2010 21:40, schrieb Jens Lehmann:
> Am 19.08.2010 21:15, schrieb Eric Wong:
>> The "--parents" option did not appear until SVN 1.5.x
>> and is completely unnecessary in this case.
> 
> Thanks for the quick response! I'll test this patch on Monday when
> I have access to that FC7 machine again.

Just for the record:

Tested-By: Jens Lehmann <Jens.Lehmann@web.de>

```
