RFR: 8072116: [Solaris] : Fix for 8071710 needs to be updated for build dependency checking

[Magnus Ihse Bursie]

On 2015-02-02 19:33, Phil Race wrote:
> http://bugs.openjdk.java.net/browse/JDK-8071710 added a depedency
> check but it was not working because this depends on file declaration 
> order.
>
> This resolves that which can cause occasional Solaris build failures by
> ensuring that HEADLESS is declared before FONTMANAGER.
>
> JDK 9 fix : http://cr.openjdk.java.net/~prr/8072116/
> JDK8u fix : http://cr.openjdk.java.net/~prr/8072116.8u
>
> FWIW I've not seen it fail on 8u but it seems like it must be possible 
> there too ..

I see now that the fix for 8u in the original JDK-8071710 did not get 
the same dependency declaration as did the 9 fix. It still says:

  948 ifneq (, $(findstring $(OPENJDK_TARGET_OS), solaris aix))
  949   $(BUILD_LIBFONTMANAGER): $(BUILD_LIBAWT_XAWT)
  950 endif

instead of depending on libawt_headless.

Apart from this, the fix looks good.

/Magnus