From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:44197)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter@lekensteyn.nl>) id 1Y8rfS-00062f-Ip
	for qemu-devel@nongnu.org; Wed, 07 Jan 2015 09:33:03 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter@lekensteyn.nl>) id 1Y8rfR-0005Y0-KY
	for qemu-devel@nongnu.org; Wed, 07 Jan 2015 09:33:02 -0500
Received: from mail.lekensteyn.nl ([2a02:2308::360:1:25]:45016)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter@lekensteyn.nl>) id 1Y8rJL-0007OQ-VJ
	for qemu-devel@nongnu.org; Wed, 07 Jan 2015 09:10:12 -0500
From: Peter Wu <peter@lekensteyn.nl>
Date: Wed, 07 Jan 2015 11:31:12 +0100
Message-ID: <1486924.zxyfEH2HUg@al>
In-Reply-To: <54AD0A80.40303@redhat.com>
References: <1419692504-29373-1-git-send-email-peter@lekensteyn.nl>
	<54AAE6CD.9080007@redhat.com> <54AD0A80.40303@redhat.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Subject: Re: [Qemu-devel] [PATCH 09/10] block/dmg: support bzip2 block entry
	types
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>, John Snow <jsnow@redhat.com>, qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@redhat.com>

On Wednesday 07 January 2015 11:29:20 Paolo Bonzini wrote:
> 
> On 05/01/2015 20:32, John Snow wrote:
> >>
> >>   fi
> >>
> >>   ##########################################
> >> +# bzip2 check
> >> +
> >> +if test "$bzip2" != "no" ; then
> >> +    cat > $TMPC << EOF
> >> +#include <bzlib.h>
> >> +int main(void) { BZ2_bzlibVersion(); return 0; }
> >> +EOF
> >> +    if compile_prog "" "-lbz2" ; then
> >> +        libs_softmmu="$libs_softmmu -lbz2"
> >> +        bzip2="yes"
> >> +    else
> >> +        if test "$bzip2" = "yes"; then
> >> +            feature_not_found "libbzip2" "Install libbzip2 devel"
> >> +        fi
> >> +        bzip2="no"
> >> +    fi
> >> +fi
> >> +
> >> +##########################################
> >>   # libseccomp check
> >>
> >>   if test "$seccomp" != "no" ; then
> >> @@ -4340,6 +4366,7 @@ echo "vhdx              $vhdx"
> >>   echo "Quorum            $quorum"
> >>   echo "lzo support       $lzo"
> >>   echo "snappy support    $snappy"
> >> +echo "bzip2 support     $bzip2"
> >>   echo "NUMA host support $numa"
> >>
> >>   if test "$sdl_too_old" = "yes"; then
> >> @@ -4695,6 +4722,10 @@ if test "$snappy" = "yes" ; then
> >>     echo "CONFIG_SNAPPY=y" >> $config_host_mak
> >>   fi
> >>
> >> +if test "$bzip2" = "yes" ; then
> >> +  echo "CONFIG_BZIP2=y" >> $config_host_mak
> >> +fi
> >> +
> >>   if test "$libiscsi" = "yes" ; then
> >>     echo "CONFIG_LIBISCSI=m" >> $config_host_mak
> >>     echo "LIBISCSI_CFLAGS=$libiscsi_cflags" >> $config_host_mak
> >>
> > 
> > Looks good otherwise. CCing Paolo so he can take a quick peek at the
> > configure script. It looks sane to me, though.
> 
> It is sane, but instead of libs_softmmu="$libs_softmmu -lbz2" it's
> better to follow what we do for libiscsi.  In configure:
> 
> CONFIG_BZIP2=y
> LIBBZ2_CFLAGS=-lbz2
> 
> In Makefile.objs:
> 
> dmg.o-libs     := $(LIBBZ2_CFLAGS)
> 
> Paolo

Already taken care of in v2 which was posted to the list yesterday :-)

I named it "BZIP2_LIBS" though (matching the package name).
-- 
Kind regards,
Peter
https://lekensteyn.nl