https://bz.apache.org/bugzilla/show_bug.cgi?id=60131

--- Comment #2 from Patrick Zimmermann <[email protected]> ---
(In reply to Nick Burch from comment #1)
> With this patch applied to trunk, but not your other patch, the unit tests
> fail with:
> 
> ...
> 
> Do we need to get your other patch fixed and applied first? Or do we need to
> fix the current tests for this patch?

I messed up a bit. Sorry for that.
Both patches together work fine.


There is a fix relating to error values in the D* database headers in here I
overlooked while preparing the patches. Because of that the new DGet.xls (which
tests that behavior) fails without this patch.

Also the DGet.xls has a false-positive test now <skip>ed which this patch
relies on.

Easiest way forward: Just apply both patches and the DGet.xls.
Alternative: I prepare another intermediate DGet.xls so one can apply the
patches one after the other.

What do you think?

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to