On Thu, Jun 18, 2009 at 6:38 AM, Hyrum K. Wright<hwright_at_juniper.net> wrote:
>
> On Jun 8, 2009, at 1:52 PM, Paul Burba wrote:
>
>> On Tue, May 26, 2009 at 11:20 AM, Hyrum K. Wright<hyrum_at_hyrumwright.org>
>> wrote:
>>>
>>> Author: hwright
>>> Date: Tue May 26 08:20:48 2009
>>> New Revision: 37825
>>>
>>> Log:
>>> Merge r37745 from trunk:
>>>
>>> Â * r37745
>>> Â Fix an *ancient* bug that permits the setting of properties on out-of-
>>> Â date directories over ra_neon when wcprops cache isn't populated.
>>> Â Justification:
>>> Â Â Modifying out of date dirprops? Â That ain't how we roll.
>>> Â Votes:
>>> Â Â +1: cmpilato, arfrever, hwright
>>>
>>> Modified:
>>> Â branches/1.6.x/ Â (props changed)
>>> Â branches/1.6.x/CHANGES Â (props changed)
>>> Â branches/1.6.x/STATUS
>>> Â branches/1.6.x/subversion/libsvn_ra_neon/commit.c
>>> Â branches/1.6.x/subversion/tests/cmdline/switch_tests.py
>>>
>>> Merged:
>>> Â /trunk:r37745
>>>
>>> Modified: branches/1.6.x/STATUS
>>> URL:
>>> http://svn.collab.net/viewvc/svn/branches/1.6.x/STATUS?pathrev=37825&r1=37824&r2=37825
>>>
>>> ==============================================================================
>>> --- branches/1.6.x/STATUS Â Â Â Tue May 26 08:19:17 2009 Â Â Â Â (r37824)
>>> +++ branches/1.6.x/STATUS Â Â Â Tue May 26 08:20:48 2009 Â Â Â Â (r37825)
>>> @@ -124,11 +124,3 @@ Approved changes:
>>> Â Improve BASH completion.
>>> Â Votes:
>>> Â Â +1: arfrever, stylesen
>>> -
>>> - * r37745
>>> - Â Fix an *ancient* bug that permits the setting of properties on
>>> out-of-
>>> - Â date directories over ra_neon when wcprops cache isn't populated.
>>> - Â Justification:
>>> - Â Â Modifying out of date dirprops? Â That ain't how we roll.
>>> - Â Votes:
>>> - Â Â +1: cmpilato, arfrever, hwright
>>>
>>> Modified: branches/1.6.x/subversion/libsvn_ra_neon/commit.c
>>> URL:
>>> http://svn.collab.net/viewvc/svn/branches/1.6.x/subversion/libsvn_ra_neon/commit.c?pathrev=37825&r1=37824&r2=37825
>>>
>>> ==============================================================================
>>> --- branches/1.6.x/subversion/libsvn_ra_neon/commit.c  Tue May 26
>>> 08:19:17 2009 Â Â Â Â (r37824)
>>> +++ branches/1.6.x/subversion/libsvn_ra_neon/commit.c  Tue May 26
>>> 08:20:48 2009 Â Â Â Â (r37825)
>>> @@ -631,10 +631,7 @@ static svn_error_t * commit_open_root(vo
>>> Â /* create the root resource. no wr_url (yet). */
>>> Â rsrc = apr_pcalloc(dir_pool, sizeof(*rsrc));
>>> Â rsrc->pool = dir_pool;
>>> -
>>> - Â /* ### should this be 'base_revision' here? we might not always be
>>> - Â Â ### working against the head! (think "properties"). */
>>> - Â rsrc->revision = SVN_INVALID_REVNUM;
>>> + Â rsrc->revision = base_revision;
>>>
>>> Â rsrc->url = cc->ras->root.path;
>>> Â rsrc->local_path = "";
>>>
>>> Modified: branches/1.6.x/subversion/tests/cmdline/switch_tests.py
>>> URL:
>>> http://svn.collab.net/viewvc/svn/branches/1.6.x/subversion/tests/cmdline/switch_tests.py?pathrev=37825&r1=37824&r2=37825
>>>
>>> ==============================================================================
>>> --- branches/1.6.x/subversion/tests/cmdline/switch_tests.py   Tue May
>>> 26 08:19:17 2009 Â Â Â Â (r37824)
>>> +++ branches/1.6.x/subversion/tests/cmdline/switch_tests.py   Tue May
>>> 26 08:20:48 2009 Â Â Â Â (r37825)
>>> @@ -2542,7 +2542,7 @@ test_list = [ None,
>>> Â Â Â Â Â Â Â relocate_beyond_repos_root,
>>> Â Â Â Â Â Â Â refresh_read_only_attribute,
>>> Â Â Â Â Â Â Â switch_change_repos_root,
>>> - Â Â Â Â Â Â Â XFail(relocate_and_propset, svntest.main.is_ra_type_dav),
>>> + Â Â Â Â Â Â Â relocate_and_propset,
>>
>> Hi Hyrum,
>>
>> This should actually be something along the lines of
>>
>>> - Â Â Â Â Â Â Â XFail(relocate_and_propset, svntest.main.is_ra_type_dav),
>>> + Â Â Â Â Â Â Â XFail(relocate_and_propset,
>>> svntest.main.is_ra_type_dav_serf),
>>
>> The revision which fixes this test over ra_serf is Mike's r36271 has
>> not been backported to 1.6.x, so this test still fails over ra_serf on
>> 1.6.x. Â One problem is that is_ra_type_dav_serf doesn't yet exist on
>> 1.6.x either, it is part of r36298. Â I'll nominate the latter in a
>> moment to make this easier.
>>
>> Paul
>>
>> P.S. Mike I'm ccing you in case r36271 *should* be nominated for
>> backport -- it would seem so).
>
> Paul,
> I didn't notice this email until *after* rolling 1.6.3, but I did see that
> there was a bunch of followup merging on 1.6.x related to this. Â Can you
> make sure that 1.6.3 is correct?
>
> -Hyrum
Welcome back Hyrum,
Sorry, I forgot about this too when I saw you were going to roll
1.6.3. I was hesitant to "fix" a backport somebody else did without
getting their approval, even though in this case it was
straightforward.
Unfortunately switch_tests.py 19 'out of date propset should fail
after a relocate' still fails over serf (as it always has for 1.6).
FWIW, if the rest of the tests pass I plan on voting +1 for the
release and simply point to this thread. I don't think a new 1.6.3 RC
is needed.
Paul
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2363184
Received on 2009-06-18 15:34:41 CEST