[PATCH:xhost 1/2] Only need CFLAGS, not LIBS from xau package
Jeremy Huddleston
jeremyhu at apple.com
Sat Sep 24 12:57:42 PDT 2011
Both:
Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>
On Sep 24, 2011, at 10:31, Alan Coopersmith wrote:
> xhost needs the Family definitions from <X11/Xauth.h> to build, but
> doesn't call any libXau functions, so doesn't need to link against it.
>
> Fixes Solaris ld -z guidance warning:
> ld: guidance: removal of unused dependency recommended: libXau.so.6
>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
> Makefile.am | 2 +-
> configure.ac | 5 ++++-
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/Makefile.am b/Makefile.am
> index be6f2c9..fbbc0f1 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -3,7 +3,7 @@ bin_PROGRAMS = xhost
>
> xhost_SOURCES = xhost.c
>
> -AM_CFLAGS = $(CWARNFLAGS) $(XHOST_CFLAGS)
> +AM_CFLAGS = $(CWARNFLAGS) $(XHOST_CFLAGS) $(XAU_CFLAGS)
> xhost_LDADD = $(XHOST_LIBS)
>
> MAINTAINERCLEANFILES = ChangeLog INSTALL
> diff --git a/configure.ac b/configure.ac
> index a30c5e2..96dde43 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -18,7 +18,10 @@ AC_TYPE_SIGNAL
> XORG_DEFAULT_OPTIONS
>
> # Checks for pkg-config packages
> -PKG_CHECK_MODULES(XHOST, x11 xmuu xau)
> +# - Both CFLAGS & LIBS needed:
> +PKG_CHECK_MODULES(XHOST, x11 xmuu)
> +# - Only CFLAGS needed, not libs:
> +PKG_CHECK_MODULES(XAU, xau)
>
> # Transport selection macro from xtrans.m4
> XTRANS_CONNECTION_FLAGS
> --
> 1.7.3.2
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list