RFR: 8149594 - Clean up Hotspot makefiles
jesper.wilhelmsson at oracle.com
Thu Feb 11 07:35:25 UTC 2016
Den 11/2/16 kl. 03:07, skrev David Holmes:
> Magnus is rewriting all of the hotspot build system. Are these cleanups really
> worthwhile at this stage?
The cleanups are worthwhile to me since I work on mostly Mac and port all my
changes over to the linux makefiles in bulk, and without these cleanups my
patches won't apply cleanly. The reason I want to push them now even though it
is a while left until the GTest stuff is done, is that every time anyone makes a
change in the makefiles (which happens more often than I had expected) I get
merge conflicts everywhere.
> 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