[8u] 8078437: Enable use of devkits for Windows.
erik.joelsson at oracle.com
Fri May 18 22:28:19 UTC 2018
On 2018-05-18 14:58, Kevin Walls wrote:
> Thanks Erik -
> OK, got those indent and line breaks added, updated webrev in the same
> Quite right, it will accept the devkit location with being told its
> specific toolchain version.
> Thanks for the feeback!
> On 18/05/2018 22:14, Erik Joelsson wrote:
>> In toolchain_windows.m4, line 296 needs indentation. Also you skipped
>> line 327-332 which were broken up in the jdk 9 change.
>> Can't see anything functionally bad, only style issues.
>> I don't think you need to specify --with-toolchain-version=2013 when
>> using the devkit, it should contain the necessary information for
>> configure to pick it up automatically.
>> On 2018-05-18 13:42, Kevin Walls wrote:
>>> I'd like to get a review of a backport from 9 to 8u:
>>> 8078437: Enable use of devkits for Windows.
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8078437
>>> 9 changeset:
>>> URL: http://hg.openjdk.java.net/jdk9/dev/rev/bc02cff96b92
>>> 9 review thread:
>>> 8u webrev: http://cr.openjdk.java.net/~kevinw/8078437/webrev.00/
>>> common/autoconf/toolchain_windows.m4 ...this one had some manual
>>> work, but some of that was just re-indenting.
>>> common/autoconf/toolchain.m4 change didn't import but the line split
>>> in the if statement looks cosmetic anyway. I copied it as I'm here.
>>> Other files import OK, plus regenerating the generated files.
>>> With this change, the regular build still works as before, plus we
>>> can do:
>>> $ bash ./configure --with-toolchain-version=2013
>>> ..and e.g. make images uses that compiler.
>>> (that does also need the hotspot change 8203349, but 2013 is the
>>> devkit bundle I have to hand)
More information about the build-dev