[PATCH libxkbcommon] Check for x11 as well through pkg-config.
Gaetan Nadon
memsize at videotron.ca
Sat Jan 29 05:02:58 PST 2011
On Sat, 2011-01-29 at 03:31 +0100, Cyril Brulebois wrote:
> Otherwise, ./configure returns successfully while xkbparse.lo can fail
> to build due to a missing include:
> | YACC xkbparse.c
> | CC xkbparse.lo
> | xkbparse.y:98:22: error: X11/Xlib.h: No such file or directory
>
> Signed-off-by: Cyril Brulebois <kibi at debian.org>
> ---
> configure.ac | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 0026441..d8db8fd 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -66,7 +66,7 @@ if test "x$CC_FOR_BUILD" = x; then
> fi
>
> # Obtain protocols headers include directives
> -PKG_CHECK_MODULES([X11], [xproto kbproto >= 1.0.5])
> +PKG_CHECK_MODULES([X11], [x11 xproto kbproto >= 1.0.5])
Reviewed-by: Gaetan Nadon <memsize at videotron.ca>
You may want to ensure this is reviewed by one the authors of this lib.
I recall some readings putting emphasis on the fact that the library did
not
depend on libX11.
This commit
http://cgit.freedesktop.org/xorg/lib/libxkbcommon/commit/?id=8d2c522f3156c3381d8530ce2346d1ec84fafe8d
says: "Don't check for xlib, we don't need it"
Maybe this include is not necessary or there is an alternate solution to
not depend on X11.
>
> # Obtain the path to the X Window System Core Protocol
> AC_MSG_CHECKING([for X11 includedir])
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110129/cc3e8f81/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110129/cc3e8f81/attachment.pgp>
More information about the xorg-devel
mailing list