On 8/28/2016 20:23, Daniel Shahaf wrote:
> Stefan wrote on Sun, Aug 28, 2016 at 13:31:39 +0200:
>> The regression test was tested against 1.9.4, 1.9.x and trunk r1743999.
>>
>> I also tried to run the test against 1.8.16 but there it fails (didn't
>> investigate in detail).
>> Trunk r1758069 caused some build issues on my machine. Therefore I
>> couldn't validate/check the patch against the latest trunk (maybe it's
>> just some local issue with my build machine rather than some actual
>> problem on trunk - didn't look into that yet).
> For future reference, you might have tried building trunk_at_HEAD after
> locally reverting r1758069; i.e.:
>
> svn up
> svn merge -c -r1758069
> <apply patch>
> make check
The problem was a local build glitch (some project files were outdated).
Unfortunately I must have run into another glitch, since after resolving
this trunk fails the new regression test with r1743999.
Will start a new thread on this one.
>> Separated the repro test from the actual fix in order to have the
>> possibility to selectively only backport the regression test to the 1.8
>> branch.
> Good call, but the fix and the "remove XFail markers" (r1758129 and
> r1758130) should have been done in a single revision: they _are_
> a single logical change. That would also avoid breaking 'make check'
> (at r1758129 'make check' exits non-zero because of the XPASS).
Right, my bad. Will take that into account the next time.
Regards,
Stefan
Received on 2016-08-28 23:03:00 CEST