RFR: JDK-8170629 Remove code duplication in test makefiles

[David Holmes]

<trimmed to build-dev and hotspot-dev>

Hi Magnus,

Some independent changes are going to collide here. :)

hotspot/make/test/JtregNative.gmk

This change has already happened in hs repo - JDK-8169261.

---

hotspot/test/Makefile

33 USE_JTREG_VERSION := 4.1

Aside: - Jigsaw bumped this to 4.2 (and I think that was a while ago). I 
don't think anything we do with jtreg relies on the JT_HOME setting 
currently set in hotspot/test/Makefile. So we should bump this after 
your refactor.

41 # hotspot currently do not use ProblemList.txt
42 NO_EXCLUDES := true

Igor is adding this right now:

http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-December/025436.html

Otherwise looks good.

Thanks,
David
-----

On 2/12/2016 7:24 PM, Magnus Ihse Bursie wrote:
> There is a lot of common code that has been duplicated in
> */test/Makefile. For jdk, hotspot, jaxp and nashorn, most of the code in
> the corresponding files are identical. (The odd-man-out is langtools
> which is quite different.)
>
> These files should be unified to share a single code base, to facilitate
> further improvements to the test makefiles.
>
> The intent of this bug is a pure refactoring. The duplication should go,
> but all functionality should remain exactly the same. This will leave
> room for some future improvements to the code, but sets a clear limit
> for this first step. The consolidated code base thus contains a fair
> amount of if-blocks, but I hope to be able to address most of these
> later on, by adjusting the individual component to adhere more to the
> standard behavior.
>
> To minimize the amount of ifdefs in the shared code, I allowed for a few
> changes in behavior. I do not believe these will cause any changes in
> the real world, and to the extent that they do, they could be considered
> bug fixes.
>
> Behavior that have changed due to unification:
> * jaxp now defaults ABS_JDK_IMAGE to images/jdk instead of j2sdk.
> * jaxp now sets TEST_SELECTION to $(TESTDIRS) if it exists.
> * jaxp and hotspot now get additional option e:JDK8_HOME=${JT_JAVA}
> * hotspot now sets JAVA_VM_ARGS to $(JPRT_PRODUCT_VM_ARGS) if it exists.
> * hotspot now sets JAVA_ARGS to $(JPRT_PRODUCT_ARGS) if it exists.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8170629
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8170629-remove-test-makefile-duplication/webrev.01
>
>
> /Magnus