[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn 1.1.x r11726: FAIL (i686-pc-linux-gnu shared ra_svn bdb)

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2004-11-03 22:22:52 CET

Philip Martin wrote:
> r11479,r11484 fix issue 1943 and add a regression test. This change
> is not on the branch.
> r11557,r11696 fix issue 1905 and add a regression test that happens to
> rely on issue 1943 behaviour. This change is on the branch so the
> test fails due to issue 1943.

Thank you for that concise and clear description.

The second pair of revisions is divided cleanly between r11557 providing the
fix and r11696 providing the regression test.

My initial feeling was that it would be cleaner to withdraw the regression test
(r11696) from the branch, because it is not essential to keep the regression
tests completely up to date in a patch-level release - but, on the other hand,
it is sensible to have tests on the branch for the bugs that have been fixed on
the branch.

Since the alternative (to disable part of the regression test on the branch) is
trivial, that seems like a good solution. I am happy now that I understand the
situation, and will leave it to you (plural) now.

(The reason I was confused and spoke up in the first place is mainly because of
the suggestion that we might want to make the same change to the trunk. I
assume it's clear that we don't want to.)

- Julian

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Nov 3 22:23:16 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.