Looks good, thanks!
/Erik
On 2018-12-10 12:23, Jakub Vaněk wrote:
Hi Erik,
On 2018-12-10 at 09:39 -0800, Erik Joelsson wrote:
The build changes look ok to me.
I do think --enable-softfloat is redundant as using any of the other
parameters would imply it to be set, but it's ok to have it there.
Thanks for the review. The motivation for this was that I was closely
following how libffi is handled. The enable option was a workaround for
the detection of softfloat in system paths. I don't think this is how
the library is going to be used, so I removed this option.
New patch can be found here:
https://github.com/ev3dev-lang-java/openjdk-ev3/blob/78a9d35986ed51273d9a3bb9878cbfcbe2488bfb/upstream/softfloat.patch
Regards,
Jakub
/Erik
< raw patch at
https://raw.githubusercontent.com/ev3dev-lang-java/openjdk-ev3/78a9d35986ed51273d9a3bb9878cbfcbe2488bfb/upstream/softfloat.patch
>
