Re: svn commit: r22757 - branches/1.4.x

On 12/19/06, dlr@tigris.org <dlr@tigris.org> wrote:

> --- branches/1.4.x/STATUS (original)
> +++ branches/1.4.x/STATUS Tue Dec 19 13:41:28 2006
> @@ -91,6 +91,7 @@
> Votes:
> +1: glasser
> +1: clkao (tested but vote not counted)
> + -0: dlr (how does r22614 figure into this?)

The nominated revision fixes the actual bug (function does not work on
transaction roots). It uses an ugly but necessary idiom that is used
in a couple other places as well.

r22610 introduces a new API (which I assume shouldn't be backported)
which eliminates the need to use that idiom; r22614 and r22629 clean
up the code in a way that only works with the new API.

I believe that backporting r22609 alone is a reasonable choice.

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org