Bug 7491

Summary: AC_PATH_XTRA requires libxt
Product: cairo Reporter: Gustavo J. A. M. Carneiro <gjc>
Component: xlib backendAssignee: Carl Worth <cworth>
Status: RESOLVED FIXED QA Contact: cairo-bugs mailing list <cairo-bugs>
Severity: normal    
Priority: high CC: chris
Version: 1.2.1   
Hardware: x86 (IA32)   
OS: Linux (All)   
i915 platform: i915 features:

Description Gustavo J. A. M. Carneiro 2006-07-11 03:13:36 UTC
Couple of complaints about configure:

1. autogen.sh is passing the option --cache-file=config.cache.  Config caching
was removed by default from autoconf for a reasons; it causes more trouble than
it's worth.  But I guess I can accept this, albeit reluctantly...

2. this is more problematic and caused me to waste a lot of time tracking it
down: the macro AC_PATH_XTRA to detect Xlibs is broken; it doesn't detect Xlibs
unless you install the libxt-dev package from the distribution.  Why cairo needs
the X toolkit (not just Xlibs) is beyond my comprehension.
Comment 1 Behdad Esfahbod 2006-07-11 06:58:19 UTC
Note that we enable configure cache only if you run autogen.sh.  Normal builds
from the tarball are not affected.
Comment 2 Behdad Esfahbod 2006-07-15 18:49:58 UTC
That may as well be a distro bug.  Donno.  I have no idea what libxt is, nor do
I have anything like that installed on my Fedora.

> Why cairo needs the X toolkit (not just Xlibs) is beyond my comprehension.

So how should cairo detect X?   And what do you mean by X toolkit?  Closing as
NOTOURBUG as we are not going to replace the autoconf-provided X detection
macros with homemade ones.
Comment 3 Gustavo J. A. M. Carneiro 2006-07-16 02:30:11 UTC
Maybe copy the Xlibs detection code from gtk+?  That one at least worked just
fine without libXt.  From what I can see, gtk+ tries to detect xlibs with
pkg-config first, and only if it fails does it fallback to AC_PATH_XTRA.

PS: libXt is an archaic X toolkit used by some core X programs such as xfontsel;
clearly cairo doesn't need it...
Comment 4 Behdad Esfahbod 2006-07-31 12:29:56 UTC
Ok, I'll look into it after we branch.  Don't want to break something
accidentally in 1.2.x.
Comment 5 Behdad Esfahbod 2006-08-18 14:17:57 UTC
Fixed.  We now look for x11 pkg-config module first.  Here is the commit:

Comment 6 Chris Wilson 2008-01-15 04:06:27 UTC
Fix released.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.