RFR: JDK-8130060: Revert use of devkit on macosx in JPRT
Tim Bell
tim.bell at oracle.com
Mon Jun 29 15:04:17 UTC 2015
Hi Erik:
> I discovered an issue with using devkits for Macosx in certain
> configurations. While it currently works fine in JPRT, I would rather
> disable it for now until the issue can be resolved.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8130060
> Patch:
> diff -r f50c05d4dcd5 make/jprt.properties
> --- a/make/jprt.properties
> +++ b/make/jprt.properties
> @@ -128,8 +128,6 @@
> ${jprt.i586.build.configure.args}
> jprt.windows_x64.build.configure.args= \
> --with-devkit=$VS2013_HOME
> -jprt.macosx_x64.build.configure.args= \
> - --with-devkit=$XCODE_511_HOME
Looks good.
Tim
> /Erik
>
> On 2015-06-26 15:53, Erik Joelsson wrote:
>> Hello,
>>
>> Please review this change for the Oracle internal JPRT configuration
>> for building jdk9. This change flips the switch for JPRT to start
>> using devkits for building jdk9 on Windows and Macosx, instead of
>> preinstalled versions of Visual Studio and Xcode. The versions of the
>> compilers stay the same for now, so nothing should really change in
>> the product. Making this change will make it much easier to change
>> compiler versions in the future.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8129969
>> Patch:
>>
>> diff -r 2f143b1e68d5 make/jprt.properties
>> --- a/make/jprt.properties
>> +++ b/make/jprt.properties
>> @@ -123,11 +123,13 @@
>> ${my.i586.default.build.configure.args} \
>> ${jprt.productOpen.build.configure.args}
>>
>> -jprt.windows_i586_6.2.build.configure.args= \
>> - --with-toolchain-version=2013 \
>> +jprt.windows_i586.build.configure.args= \
>> + --with-devkit=$VS2013_HOME \
>> ${jprt.i586.build.configure.args}
>> -jprt.windows_x64_6.2.build.configure.args= \
>> - --with-toolchain-version=2013
>> +jprt.windows_x64.build.configure.args= \
>> + --with-devkit=$VS2013_HOME
>> +jprt.macosx_x64.build.configure.args= \
>> + --with-devkit=$XCODE_511_HOME
>>
>> ########
>> #
>>
>
More information about the build-dev
mailing list