RFR (S): 8180453: mx eclipseinit doesn't pick up generated sources
Christian Thalinger
cthalinger at twitter.com
Tue May 16 23:32:51 UTC 2017
> On May 16, 2017, at 1:29 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>
> JDK 9 or JDK 10?
Both.
>
> Changes are fine but you need approval for JDK 9.
How? Bunch of labels, I guess? :-)
>
> Thanks,
> Vladimir
>
> On 5/16/17 4:17 PM, Christian Thalinger wrote:
>> https://bugs.openjdk.java.net/browse/JDK-8180453 <https://bugs.openjdk.java.net/browse/JDK-8180453>
>>
>> The fix is trivial and does not affect production code.
>>
>> diff --git a/.mx.jvmci/mx_jvmci.py b/.mx.jvmci/mx_jvmci.py
>> index b87bab7..37a9baf 100644
>> --- a/.mx.jvmci/mx_jvmci.py
>> +++ b/.mx.jvmci/mx_jvmci.py
>> @@ -303,9 +303,9 @@ class HotSpotProject(mx.NativeProject):
>> out.close('link')
>>
>> out.open('link')
>> - out.element('name', data='generated')
>> + out.element('name', data='gensrc')
>> out.element('type', data='2')
>> - generated = join(_get_hotspot_build_dir(jvmVariant, debugLevel), 'generated')
>> + generated = join(_get_hotspot_build_dir(jvmVariant, debugLevel), 'gensrc')
>> out.element('locationURI', data=mx.get_eclipse_project_rel_locationURI(generated, eclProjectDir))
>> out.close('link')
>>
>> @@ -620,18 +620,12 @@ _jvmci_bootclasspath_prepends = []
>> def _get_hotspot_build_dir(jvmVariant=None, debugLevel=None):
>> """
>> Gets the directory in which a particular HotSpot configuration is built
>> - (e.g., <JDK_REPO_ROOT>/build/macosx-x86_64-normal-server-release/hotspot/bsd_amd64_compiler2)
>> + (e.g., <JDK_REPO_ROOT>/build/macosx-x86_64-normal-server-release/hotspot/variant-<variant>)
>> """
>> if jvmVariant is None:
>> jvmVariant = _vm.jvmVariant
>>
>> - os = mx.get_os()
>> - if os == 'darwin':
>> - os = 'bsd'
>> - arch = mx.get_arch()
>> - buildname = {'client': 'compiler1', 'server': 'compiler2'}.get(jvmVariant, jvmVariant)
>> -
>> - name = '{}_{}_{}'.format(os, arch, buildname)
>> + name = 'variant-{}'.format(jvmVariant)
>> return join(_get_jdk_build_dir(debugLevel=debugLevel), 'hotspot', name)
>>
>> class JVMCI9JDKConfig(mx.JDKConfig):
>>
More information about the hotspot-dev
mailing list