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

2016-08-04 Thread David Holmes
On 4/08/2016 6:24 PM, Erik Joelsson wrote: Hello, New webrev: http://cr.openjdk.java.net/~erikj/8163102/webrev.02 The difference is in libraries.m4, where there was still a reference to SUPPORT_HEADFUL. I tried building a headless configuration without X11 configured, but it unfortunately faile

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

2016-08-04 Thread Erik Joelsson
Hello, New webrev: http://cr.openjdk.java.net/~erikj/8163102/webrev.02 The difference is in libraries.m4, where there was still a reference to SUPPORT_HEADFUL. I tried building a headless configuration without X11 configured, but it unfortunately failed. libawt.so still requires X11 headers t

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

2016-08-03 Thread David Holmes
Hi Erik, On 4/08/2016 1: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. The confusion around that variable due to its legacy alw

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

2016-08-03 Thread 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

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

2016-08-03 Thread Tim Bell
Erik: 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 bot

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

2016-08-03 Thread Erik Joelsson
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 bo