Bug 7491

Summary:	AC_PATH_XTRA requires libxt
Product:	cairo	Reporter:	Gustavo J. A. M. Carneiro <gjc>
Component:	xlib backend	Assignee:	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)
Whiteboard:
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:

  0e78e7144353703cbd28aae6a67cd9ca261f1d68

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.