On Thu, Nov 15, 2018 at 5:55 PM Philip Race <philip.r...@oracle.com> wrote: > > Hmm. I don't like the ifdefs. >
Mee too, but what else can we do, if there are no Xrandr headers available on a platform? Please see my answer to your other mail... > Xrandr is a requirement for the build. If its not there at runtime > that's OK. > > -phil. > > On 11/15/18, 8:06 AM, Volker Simonis wrote: > > Hi, > > > > can I please have a review for the following small change: > > > > http://cr.openjdk.java.net/~simonis/webrevs/2018/8213944/ > > https://bugs.openjdk.java.net/browse/JDK-8213944 > > > > Change JDK-8210863 removed the Xrandr.h/randr.h headers from the > > OpenJDK sources but forgot to add a configure check for the Xrandr > > extension which is now a build dependency. > > > > The change also broke the AIX build. AIX never supported Xrandr, but > > that was only detected at runtime, when the JDK was unable to > > dynamically load libXrand.so. Now, without Xrandr.h/randr.h in the > > source tree any more, we have to conditionally compile some parts of > > src/java.desktop/unix/native/libawt_xawt/awt/awt_GraphicsEnv.c such > > that it doesn't require the definitions and declarations from > > Xrandr.h/randr.h any more. > > > > Thank you and best regards, > > Volker