Hi Volker, I had a look at your change. It looks good. I appreciate a lot you added a check in configure. Maybe it would be better to pass a WITHOUT_XRANDR or the like to the build, and check for such a define in the code. But I think we should push this change for now to fix the build. A follow up still can re-enable xrandr on AIX again or prettify the code.
Best regards, Goetz. > -----Original Message----- > From: 2d-dev <2d-dev-boun...@openjdk.java.net> On Behalf Of Philip Race > Sent: Thursday, November 15, 2018 6:02 PM > To: Volker Simonis <volker.simo...@gmail.com> > Cc: 2d-dev <2d-...@openjdk.java.net>; build-dev <build- > d...@openjdk.java.net> > Subject: Re: [OpenJDK 2D-Dev] RFR(XS): 8213944: Fix AIX build after the > removal of Xrandr.h and add a configure check for it > > PS I am not sure why xrandr headers would not be available for AIX. > They are a standard part of the xdistribution. > > If true, think what you are going to have to do is add a > --with-xrandr-include option > and provide it that way. > > -phil. > > On 11/15/18, 8:55 AM, Philip Race wrote: > > Hmm. I don't like the ifdefs. > > > > 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