RFR (S): 7178667: ALT_EXPORT_PATH does not export server jvm on macosx

Dmitry Samersoff Dmitry.Samersoff at oracle.com
Wed Jun 27 03:03:01 PDT 2012


Looks good for me.

-Dmitry

On 2012-06-27 12:13, Staffan Larsen wrote:
> Can I have a Review for this change, please?
> 
> The very simple fix is
> here: http://cr.openjdk.java.net/~sla/7178667/webrev.02/
> 
> Thanks,
> /Staffan
> 
> On 25 jun 2012, at 10:36, Staffan Larsen wrote:
> 
>>
>>>>> So, it sounds like when you rebuilt, everything was built into
>>>>> jre/lib/i386 and jre/lib/amd64, but never combined (or, in this
>>>>> case, just copied) into jre/lib, and therefore not found.
>>>>
>>>> Yes. Or rather, only the client jvm was combined, but the client jvm
>>>> isn't copied into the j2sdk-image on mac, so nothing was copied.
>>>
>>> Which begs the question: if we only build 64-bit on OSX then how/why
>>> is client being built in the first place?
>>
>> I should have said: "only the client jvm was _attempted_ to be
>> combined". In fact, the client does not exist, but the universalize
>> makefiles are written to handle client if it did exist.
>>
>> So what happened was:
>> - the product jvm was built
>> - it was copied to the import jdk (into jre/lib/amd64/server/) by the
>> generic_export target
>> - the universalize makefile tried to take the client jvm and
>> universalize it into jre/lib/client/ (notice that there is no amd64
>> directory level on mac)
>> - the universalize makefile removes all {amd64,i386} directories
>>
>> What should have happened:
>> - the product jvm was built
>> - it was copied to the import jdk (into jre/lib/amd64/server/) by the
>> generic_export target
>> - the universalize makefile makes a universal binary of any existing
>> jvms (client or server)
>> - the universalize makefile copies these jvms into jre/lib/{server,client}
>> - the universalize makefile removes all {amd64,i386} directories
>>
>> But because the targets weren't .PHONY, the third step above failed.
>>
>> I hope that explains the problem in more detail. Who wants to be put
>> down as reviewer?
>>
>> Thanks,
>> /Staffan
>>
> 


-- 
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...




More information about the hotspot-runtime-dev mailing list