RFR: JDK-8203221 Makefile fixes after Flight Recorder

[Magnus Ihse Bursie]

> 16 maj 2018 kl. 01:39 skrev Erik Joelsson <erik.joelsson at oracle.com>:
> 
> Hello,
> 
> In GensrcJfr, JFR_TOOLS_OUTPUTDIR is defined twice.

Oops, will fix. 

> Other build tools are in make/{jdk,hotspot}/src/classes. Do you think we should be moving them to one place? Regardless, I think we need an INCLUDE in the SetupJavaCompilation so that we only build the tool we need. There is currently no other tools in make/src/classes, but that directory is inviting for others.

I think we should move all tools to a common place. The current solution is more an effect of how things ended up due to the old multi-repo design. 

So I though: let's start doing things right, and put this one in a common build tool directory, and then we can move the rest there in a follow up.

But if you think that is confusing (I could really agree) we can skip doing that now. If so, I'll just move this to some jfr_build_tools_classes dir instead. 

/Magnus

> 
> /Erik
> 
> 
>> On 2018-05-15 16:14, Magnus Ihse Bursie wrote:
>> Cleanups of the build system after Flight Recorder.
>> 
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8203221
>> WebRev: http://cr.openjdk.java.net/~ihse/JDK-8203221-makefile-fixes-after-flight-recorder/webrev.01
>> 
>> /Magnus
>