On Fri, Oct 2, 2009 at 7:03 PM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> Bert Huijben wrote:
>>
>> > Author: julianfoad
>> > Date: Wed Sep 30 10:57:51 2009
>> > New Revision: 39716
>> >
>> > Log:
>> > Make the test suite better about choosing between RA-Neon and RA-Serf.
>> > Update the RA type detection functions to reflect that the default is
>> > "serf",
>> > so that they work when "--http-library" is not specified. Give the test
>> > suite's "--http-library" option its own default value ("serf") so that
>> > it no longer depends on svn's compiled-in default value.
>> >
>> > * subversion/tests/cmdline/svntest/main.py
>> > (http_library): Rename this variable to preferred_http_library
>> > throughout.
>> > Clarify its doc string.
>> > (is_not_serf): Remove this unused function.
>> > (is_ra_type_dav_neon, is_ra_type_dav_serf): No longer assume the
>> > default is
>> > "neon". Add doc strings.
>> > (is_ra_type_svn, is_ra_type_file): Add doc strings.
>> > (usage): Mention the default value of --http-library in the help
>> > message.
>>
>> For some reason it seems this change fails two tests on the neon/serf
>> buildbots.
>>
>> Neon:
>> XPASS: merge_tests.py 132: replace vs. edit tree-conflicts
>>
>> And Serf:
>> FAIL: merge_authz_tests.py 1: skipped paths get overriding mergeinfo
>>
>> The first has an:
>> XFail(merge_replace_causes_tree_conflict,
>> svntest.main.is_ra_type_dav_serf),
>> marking and the second a
>> SkipUnless(Skip(mergeinfo_and_skipped_paths, svntest.main.is_ra_type_file),
>> svntest.main.server_has_mergeinfo),
>
> I think I have worked out what is wrong.
>
> Before my change, the test suite's "--http-library" option defaulted to
> "neon". Back in the 1.6 days and earlier, that was sensible.
>
> Because Serf is now the default, I changed the test suite's
> "--http-library" option to default to "serf", and also made the test
> suite always tell Subversion (via a config option) to prefer to use
> whichever library the test suite was told to use. I thought that made
> sense.
>
> What's happening here, I think, is the macosx/neon build-bot is building
> Subversion with Neon only, and passing no "--http-library" option to the
> test suite. I've changed the default assumption so it now gets it wrong.
Yup, that's correct. The two buildbots running on the mac have either
ra_neon or ra_serf, not both at the same time.
Lieven
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2403005
Received on 2009-10-02 19:17:26 CEST