[Spice-devel] [PATCH v2] cleanup x11 library detection for building client
Christophe Fergeau
cfergeau at redhat.com
Fri May 4 06:10:06 PDT 2012
It's now pushed at
http://cgit.freedesktop.org/spice/spice/commit/?id=c3eae628725012678b7141b49b0a15b8718ea9b3
Thanks again!
Christophe
On Fri, May 04, 2012 at 10:25:03AM +0200, Christophe Fergeau wrote:
> Hey Nahum,
>
> On Thu, May 03, 2012 at 12:32:19PM -0400, Nahum Shalman wrote:
> > Consolidate two separate chunks of library hunting that depend on the
> > same check.
> >
> > Check if we're actually building the client before looking for
> > client only libraries.
> >
> > Hide some of the final output if we're not building the client.
> > ---
> > I think this should address the rest of the concerns about
> > the previous version of this patch.
>
> Patch looks good to me, and worked with and without --disable-client. I'll
> push it later today, just waiting a little bit in case someone has more
> comments. I'll fold the patch below when pushing to remove some unwanted
> blank lines in the summary.
>
> Thanks for sending this patch and addressing my comments!
>
> Christophe
>
>
> diff --git a/configure.ac b/configure.ac
> index 6853d05..66f9d12 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -482,7 +482,7 @@ echo "
> python: ${PYTHON}
>
> Build Spice client: ${enable_client}
> -" ; if test "x$enable_client" == "xyes"; then echo "
> +" ; if test "x$enable_client" == "xyes"; then echo "\
> Have XRANDR 1.2: ${have_xrandr12}
>
> Have Xinerama: ${have_xinerama}
> @@ -492,7 +492,7 @@ echo "
> OpenGL: ${enable_opengl}
>
> GUI: ${enable_gui}
> -" ; fi ; echo "
> +" ; fi ; echo "\
> Support tunneling: ${enable_tunnel}
>
> Smartcard: ${enable_smartcard}
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- 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/20120504/c6245090/attachment.pgp>
More information about the Spice-devel
mailing list