[OpenJDK 2D-Dev] RFR: JDK-8163102: Fix headless only configuration option

[Philip Race]

Erik,

David is already cleaning up the jawt.c file here :-
http://cr.openjdk.java.net/~dholmes/8140723/webrev.jdk/src/java.desktop/unix/native/libjawt/jawt.c.sdiff.html

Not sure who should win :-)

-phil.

On 8/3/16, 8:04 AM, Erik Joelsson wrote:
> Hello,
>
> The current --disable-headful option is weird. Even worse, it doesn't 
> work since the makefiles still uses the legacy BUILD_HEADLESS_ONLY 
> variable, which configure never touches.
>
> In this patch I'm replacing the configure option with 
> --enable-headless-only. Default is disabled, where both headless and 
> GUI support is built. When enabled, only headless support is built. 
> These are the two variants we are ever interested in.
>
> On the makefile side, I redefined the variable as 
> ENABLE_HEADLESS_ONLY=true/false.
>
> I also needed to change jawt.c to get a headless only build to 
> actually work. The requirement that -DJAVASE_EMBEDDED is set seems 
> unnecessary and unrelated to me.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8163102
>
> Webrev: http://cr.openjdk.java.net/~erikj/8163102/webrev.01
>
> /Erik
>