[Spice-devel] [PATCH usbclerk 4/6] build-sys: link with libusbredir

Christophe Fergeau cfergeau at redhat.com
Tue Feb 19 10:08:35 PST 2013


On Tue, Feb 19, 2013 at 11:35:29AM +0100, Marc-André Lureau wrote:
> There is no need to copy files, we have the lib already, and we can
> linking statically.
> ---
>  Makefile.am      |   4 +-
>  configure.ac     |   5 +-
>  stdint.h         | 256 ------------------------------------------------------
>  usbclerk.vcproj  |  40 ---------
>  usbredirfilter.c | 260 -------------------------------------------------------
>  usbredirfilter.h | 127 ---------------------------
>  6 files changed, 5 insertions(+), 687 deletions(-)
>  delete mode 100644 stdint.h
>  delete mode 100644 usbredirfilter.c
>  delete mode 100644 usbredirfilter.h
> 
> diff --git a/Makefile.am b/Makefile.am
> index 77cf7a1..bb28145 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -3,8 +3,8 @@ NULL =
>  
>  bin_PROGRAMS = usbclerk usbclerktest
>  
> -usbclerk_LDFLAGS = $(LIBWDI_LIBS) -lversion -all-static
> -usbclerk_CPPFLAGS = $(LIBWDI_CFLAGS)
> +usbclerk_LDFLAGS = $(USBCLERK_LIBS) -lversion -lsetupapi -all-static
> +usbclerk_CPPFLAGS = $(USBCLERK_CFLAGS)
>  usbclerk_SOURCES = usbclerk.cpp vdlog.cpp usbclerk.h vdlog.h
>  
>  usbclerktest_LDFLAGS =
> diff --git a/configure.ac b/configure.ac
> index aafb51c..c6c9329 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -11,8 +11,9 @@ AM_MAINTAINER_MODE([enable])
>  LT_INIT
>  AC_PROG_CXX
>  
> -PKG_CHECK_MODULES(LIBWDI, libwdi)
> -LIBWDI_LIBS=`$PKG_CONFIG --static --libs libwdi`
> +USBCLERK_DEPS="libwdi libusbredirparser-0.5 >= 0.6"
> +PKG_CHECK_MODULES(USBCLERK, $USBCLERK_DEPS)
> +LIBWDI_LIBS=`$PKG_CONFIG --static --libs $USBCLERK_DEPS`
>  
>  m4_ifdef([AM_SILENT_RULES],[AM_SILENT_RULES([yes])])
  

the usbredir* files don't seem to be used in the mingw build, even before
this patch. I suggest the Makefile.am/configure.ac bits are moved to the
first patch, and that this one is just "remove usbredir* files", though I'd
tend to try to keep things building on vc++ as well as long as it's clear
that this is not actively maintained.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20130219/25b9feee/attachment.pgp>


More information about the Spice-devel mailing list