GNOME Bugzilla – Bug 314079
configure.in doesnt check for Xauth existence and usability
Last modified: 2006-11-13 21:13:37 UTC
Due to the dependence from gnome-session on Xauth, configure.in must test if this library is installed in the system. When I was building gnome-session, the build failed due to a undefined reference to XauDisposeAuth. The problem was that I haven't had Xauth library installed, but the configure script didn't notice it.
Created attachment 51059 [details] [review] Proposed patch. Simple check for the existence of the header file.
You could improve the error message: "You must install the Xauth library and headers to compile gnome-session." or something along those lines. Vincent, can this go in with that change?
No problem. I can improve it. Just waiting for Vincent comment.
FWIW this patch (with an update to the text) looks reasonable to me. (In general it doesn't make sense to check for required headers -- IMO having the build fail is just as good as having configure fail. But I don't think it matters much either way...)
I committed a similar patch.
The patch is erroneous. It's missing a final parenthesis. ])]) instead of )]]
Another problem I have is that configure doesn't look for libXau in "/usr/X11/lib" and I'll not pass this test. I guess a LIBS variable is missing.
Reopening; see comment #6 and comment #7.
*** Bug 362050 has been marked as a duplicate of this bug. ***
Bug #362050 has the same fix as a patch, fwiw.
*** Bug 362559 has been marked as a duplicate of this bug. ***
Was fixed by the commit for bug #362541. *** This bug has been marked as a duplicate of 362541 ***
comment #7 still applies; http://jhbuild.bxlug.be/builds/2006-11-01-0000/logs/gnome-session/#configure (patch to follow)
Created attachment 75785 [details] [review] use X linking flags, so Xauth in /usr/X11R6/libs/ will be found. use X linking flags, so Xauth in /usr/X11R6/libs/ will be found.
Thanks.