https://issues.apache.org/ooo/show_bug.cgi?id=118906

--- Comment #5 from Pedro Giffuni <[email protected]> ---
Hi Andre;

To be honest, all I did was to review the lp_solve removal and adapt it to
coinmp. The tests for Ubunto were there, along with an attempt to grok a
function in the library. I think we can leave them out and wait for someone
with Ubuntu to complain ;).

Concerning the check, Lp_solve seemed to take the decision in calc/makefile.mk.
I am unsure if I shouldve used an "else" there. Further checks are certainly
better but I couldnt find them in the deprecated lp_solve.

The patch is still untested: I have to do other changes in configure.in for
binary blobs and I am yet undecided how to go around testing all of it since
builds take so long.

Once its done, this patch should also be a good guideline to do the same with
apr and serf, of course.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to