[PATCH:libXcomposite] Remove unneeded dependencies from configure.ac & xcomposite.pc

Peter Hutterer peter.hutterer at who-t.net
Sun Jul 4 16:37:14 PDT 2010


On Wed, Jun 30, 2010 at 10:53:33PM -0700, Alan Coopersmith wrote:
> We only need Xfixes headers for the XserverRegion definition, don't
> need to link against it directly or require clients link against it,
> so only use CFLAGS from it in the Makefile, and only list it in
> Requires.private in the .pc file.
> 
> Don't need libXext at all - no headers or functions from it are used.
> 
> Also clean up some extra appearances of *_CFLAGS & *_LIBS that aren't needed.
> 
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  configure.ac     |    5 ++---
>  src/Makefile.am  |    4 ++--
>  xcomposite.pc.in |    6 +++---
>  3 files changed, 7 insertions(+), 8 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 3f44a9d..e63ef8f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -65,9 +65,8 @@ if test "$VERSION" = "" ; then
>  fi
>  COMPOSITEEXT_VERSION=[`echo $VERSION | sed 's/^\([0-9][0-9]*\.[0-9][0-9]*\).*$/\1/'`]
>  AC_SUBST(COMPOSITEEXT_VERSION)
> -PKG_CHECK_MODULES(XCOMPOSITE, [compositeproto >= $COMPOSITEEXT_VERSION] x11 xfixes xext fixesproto)
> -AC_SUBST(XCOMPOSITE_CFLAGS)
> -AC_SUBST(XCOMPOSITE_LIBS)
> +PKG_CHECK_MODULES(XCOMPOSITE, [compositeproto >= $COMPOSITEEXT_VERSION] x11)
> +PKG_CHECK_MODULES(XFIXES, xfixes)
>  
>  AC_OUTPUT([Makefile
>  	   src/Makefile
> diff --git a/src/Makefile.am b/src/Makefile.am
> index e7b9db8..456bb88 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -19,7 +19,7 @@
>  #  TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
>  #  PERFORMANCE OF THIS SOFTWARE.
>  
> -AM_CFLAGS = $(CWARNFLAGS) $(XCOMPOSITE_CFLAGS) $(X_CFLAGS)
> +AM_CFLAGS = $(CWARNFLAGS) $(XCOMPOSITE_CFLAGS) $(XFIXES_CFLAGS)
>  AM_CPPFLAGS = -I$(top_srcdir)/include
>  
>  lib_LTLIBRARIES = libXcomposite.la
> @@ -28,7 +28,7 @@ libXcomposite_la_SOURCES = \
>  	xcompositeint.h	\
>  	Xcomposite.c
>  
> -libXcomposite_la_LIBADD = $(XCOMPOSITE_LIBS) $(X_LIBS) $(XCOMPOSITE_LIBS)
> +libXcomposite_la_LIBADD = $(XCOMPOSITE_LIBS)
>  
>  libXcomposite_la_LDFLAGS = -version-number 1:0:0 -no-undefined
>  
> diff --git a/xcomposite.pc.in b/xcomposite.pc.in
> index 18c2a12..bed1f29 100644
> --- a/xcomposite.pc.in
> +++ b/xcomposite.pc.in
> @@ -4,9 +4,9 @@ libdir=@libdir@
>  includedir=@includedir@
>  
>  Name: Xcomposite
> -Description: X Composite  Library
> +Description: X Composite Extension Library
>  Version: @PACKAGE_VERSION@
> -Requires: xproto compositeproto >= @COMPOSITEEXT_VERSION@ xfixes
> -Requires.private: x11 xfixes xext
> +Requires: xproto compositeproto >= @COMPOSITEEXT_VERSION@
> +Requires.private: x11 xfixes
>  Cflags: -I${includedir}
>  Libs: -L${libdir} -lXcomposite
> -- 
> 1.5.6.5
> 

Acked-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
  Peter


More information about the xorg-devel mailing list