RFR: 8149594 - Clean up Hotspot makefiles
david.holmes at oracle.com
Thu Feb 11 02:07:49 UTC 2016
Magnus is rewriting all of the hotspot build system. Are these cleanups
really worthwhile at this stage?
On 11/02/2016 8:10 AM, Jesper Wilhelmsson wrote:
> Sending again to include the build-dev list.
> Den 10/2/16 kl. 21:31, skrev Jesper Wilhelmsson:
>> Please review this cleanup of the Hotspot makefiles.
>> Since I have been spending some time in the makefiles lately there
>> were a few
>> random cleanups that I couldn't stop myself from doing. Most of these
>> are made
>> to make the linux and bsd makefiles more alike. This has helped a lot
>> porting the framework to the different platforms.
>> There are a couple of preparing alignment changes that I included in this
>> cleanup to make the Google test patch easier to review later.
>> There are also a couple of "real" changes:
>> * In make/bsd/makefiles/buildtree.make we set up OS_VENDOR with the
>> that we don't include defs.make. Three lines below we include defs.make.
>> * In make/bsd/makefiles/buildtree.make the 'install' target depends on
>> 'install_jsigs'. There is no rule called 'install_jsigs', it is called
>> Another difference that I find interesting but that I have not changed
>> in this
>> patch (I can do that if requested) is that in the bsd version of
>> VERSION is set to "fastdebug" but in the linux version it is set to
>> Given the name of the makefile fastdebug seems more correct, but
>> whichever is
>> the correct value, shouldn't they be the same on linux and bsd?
More information about the build-dev