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

RE: Windows buildbot failures

From: Bert Huijben <rhuijben_at_sharpsvn.net>
Date: Fri, 3 Apr 2009 13:43:02 +0200

> -----Original Message-----
> From: Bert Huijben [mailto:rhuijben_at_sharpsvn.net]
> Sent: donderdag 2 april 2009 22:47
> To: dev_at_subversion.tigris.org
> Subject: Windows buildbot failures
>
> Hi,
>
> Thanks to Paul the reporting of the Windows tests is fixed, so we are now
> back to the following list of failed tests (on the Windows buildbot and
> confirmed here locally):
>
> At least one test FAILED, checking M:\svn-auto-test\fsfs\tests.log
> FAIL: revert_tests.py 1: revert relative to wc root
> FAIL: patch_tests.py 1: 'svn patch' basic functionality with no unidiff
> FAIL: patch_tests.py 2: apply a unidiff patch -- test external tool
> FAIL: patch_tests.py 3: test copy and move operations
>
> I think those last 3 are still related to the merging of the svn-patch
> branch?
> The XFail of these tests was removed in r36564 (two weeks ago). I don't
> think the patch detection works corrently on these tests.

These 3 tests are skipped if no patch is found, but the Windows buildbot and
the PC I used for testing both have a patch.exe.

The issue I see now is that the created patches (in the patch test) have
"\r\n" (Windows style) newlines and it looks like our svn patch expects "\n"
(Unix style) newlines.

Changing the newlines of the generated patch to "\n" resolved the "patch:
**** Only garbage was found in the patch input." error when used outside the
test framework, but I think there are more problems than just this one.

I think we should at least allow "\r\n" newlines in the non-unified-patch
blocks (The 'SVNPATCH1 BLOCK' blocks) on systems that don't default to "\n".

        Bert
>
> The revert test is more recent and might be related to the wc-ng work or
> changes in the testing framework.
> (See http://www.mobsol.be/buildbot/waterfall for the details).
>
> It would be nice if we can get the buildbot green tomorrow (For the first
> time in about 3 weeks).
>
> Some help on investigating revert_tests.py would be welcome...
> (A quick scan of the buildbot logs says the failure was introduced
somewhere
> between 36879 and r36895.)
>
> Thanks,
> Bert
>
> ------------------------------------------------------
>
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=152
40
> 86

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1531587
Received on 2009-04-03 13:51:28 CEST

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.