Message ID | 1436354392-26357-1-git-send-email-pbonzini@redhat.com |
---|---|
State | New |
Headers | show |
On 8 July 2015 at 12:19, Paolo Bonzini <pbonzini@redhat.com> wrote: > The following changes since commit f6e3035f75e5c6a73485335765ae070304c7a110: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream-smm' into staging (2015-07-06 23:37:53 +0100) > > are available in the git repository at: > > git://github.com/bonzini/qemu.git tags/for-upstream > > for you to fetch changes up to 2828a307232ffceeddec9feb6a87ac660b68b693: > > ossaudio: fix memory leak (2015-07-08 13:11:01 +0200) > > ---------------------------------------------------------------- > Bugfixes and Daniel Berrange's crypto library. I'm afraid this fails to link on my linux-static build: c++ -I/usr/include/pixman-1 -Werror -m64 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -Wendif-labels -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits -fstack-protector-all -I/usr/include/p11-kit-1 -I/usr/include/libpng12 -I/usr/include/libusb-1.0 -I/home/petmay01/linaro/qemu-for-merges/tests -pthread -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -g -Wl,--warn-common -m64 -static -g -o qemu-nbd qemu-nbd.o async.o thread-pool.o nbd.o block.o blockjob.o main-loop.o iohandler.o qemu-timer.o aio-posix.o qemu-io-cmds.o qemu-coroutine.o qemu-coroutine-lock.o qemu-coroutine-io.o qemu-coroutine-sleep.o coroutine-ucontext.o block/raw_bsd.o block/qcow.o block/vdi.o block/vmdk.o block/cloop.o block/bochs.o block/vpc.o block/vvfat.o block/qcow2.o block/qcow2-refcount.o block/qcow2-cluster.o block/qcow2-snapshot.o block/qcow2-cache.o block/qed.o block/qed-gencb.o block/qed-l2-cache.o block/qed-table.o block/qed-cluster.o block/qed-check.o block/vhdx.o block/vhdx-endian.o block/vhdx-log.o block/quorum.o block/parallels.o block/blkdebug.o block/blkverify.o block/block-backend.o block/snapshot.o block/qapi.o block/raw-posix.o block/linux-aio.o block/null.o block/mirror.o block/io.o block/throttle-groups.o block/nbd.o block/nbd-client.o block/sheepdog.o block/accounting.o block/write-threshold.o block/dmg.o libqemuutil.a libqemustub.a -lz -laio -lm -pthread -lgthread-2.0 -lglib-2.0 -lpcre -lz -lrt -lz -lcap-ng -lvdeplug -luuid -lgcrypt -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt -lgpg-error -ltasn1 -lz -lp11-kit -lutil c++: error: unrecognized command line option ‘-R’ (That's configure arguments '--cc=ccache gcc' '--enable-debug' '--static' '--disable-system', on an x86-64 Linux host.) -- PMM
On 08/07/2015 16:46, Peter Maydell wrote: > I'm afraid this fails to link on my linux-static build: > > c++ -I/usr/include/pixman-1 -Werror -m64 -D_GNU_SOURCE > -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes > -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes > -fno-strict-aliasing -fno-common -Wendif-labels > -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security > -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration > -Wold-style-definition -Wtype-limits -fstack-protector-all > -I/usr/include/p11-kit-1 -I/usr/include/libpng12 > -I/usr/include/libusb-1.0 > -I/home/petmay01/linaro/qemu-for-merges/tests -pthread > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include > -g -Wl,--warn-common -m64 -static -g -o qemu-nbd qemu-nbd.o async.o > thread-pool.o nbd.o block.o blockjob.o main-loop.o iohandler.o > qemu-timer.o aio-posix.o qemu-io-cmds.o qemu-coroutine.o > qemu-coroutine-lock.o qemu-coroutine-io.o qemu-coroutine-sleep.o > coroutine-ucontext.o block/raw_bsd.o block/qcow.o block/vdi.o > block/vmdk.o block/cloop.o block/bochs.o block/vpc.o block/vvfat.o > block/qcow2.o block/qcow2-refcount.o block/qcow2-cluster.o > block/qcow2-snapshot.o block/qcow2-cache.o block/qed.o > block/qed-gencb.o block/qed-l2-cache.o block/qed-table.o > block/qed-cluster.o block/qed-check.o block/vhdx.o block/vhdx-endian.o > block/vhdx-log.o block/quorum.o block/parallels.o block/blkdebug.o > block/blkverify.o block/block-backend.o block/snapshot.o block/qapi.o > block/raw-posix.o block/linux-aio.o block/null.o block/mirror.o > block/io.o block/throttle-groups.o block/nbd.o block/nbd-client.o > block/sheepdog.o block/accounting.o block/write-threshold.o > block/dmg.o libqemuutil.a libqemustub.a -lz -laio -lm -pthread > -lgthread-2.0 -lglib-2.0 -lpcre -lz -lrt -lz -lcap-ng -lvdeplug > -luuid -lgcrypt -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt > -lgpg-error -ltasn1 -lz -lp11-kit -lutil > c++: error: unrecognized command line option ‘-R’ > > (That's configure arguments '--cc=ccache gcc' '--enable-debug' > '--static' '--disable-system', on an x86-64 Linux host.) You don't have a static gnutls installed, you need to add either --disable-gnutls or --disable-tools. Paolo
On 8 July 2015 at 16:21, Paolo Bonzini <pbonzini@redhat.com> wrote: > > > On 08/07/2015 16:46, Peter Maydell wrote: >> I'm afraid this fails to link on my linux-static build: >> >> c++ -I/usr/include/pixman-1 -Werror -m64 -D_GNU_SOURCE >> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes >> -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes >> -fno-strict-aliasing -fno-common -Wendif-labels >> -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security >> -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration >> -Wold-style-definition -Wtype-limits -fstack-protector-all >> -I/usr/include/p11-kit-1 -I/usr/include/libpng12 >> -I/usr/include/libusb-1.0 >> -I/home/petmay01/linaro/qemu-for-merges/tests -pthread >> -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include >> -g -Wl,--warn-common -m64 -static -g -o qemu-nbd qemu-nbd.o async.o >> thread-pool.o nbd.o block.o blockjob.o main-loop.o iohandler.o >> qemu-timer.o aio-posix.o qemu-io-cmds.o qemu-coroutine.o >> qemu-coroutine-lock.o qemu-coroutine-io.o qemu-coroutine-sleep.o >> coroutine-ucontext.o block/raw_bsd.o block/qcow.o block/vdi.o >> block/vmdk.o block/cloop.o block/bochs.o block/vpc.o block/vvfat.o >> block/qcow2.o block/qcow2-refcount.o block/qcow2-cluster.o >> block/qcow2-snapshot.o block/qcow2-cache.o block/qed.o >> block/qed-gencb.o block/qed-l2-cache.o block/qed-table.o >> block/qed-cluster.o block/qed-check.o block/vhdx.o block/vhdx-endian.o >> block/vhdx-log.o block/quorum.o block/parallels.o block/blkdebug.o >> block/blkverify.o block/block-backend.o block/snapshot.o block/qapi.o >> block/raw-posix.o block/linux-aio.o block/null.o block/mirror.o >> block/io.o block/throttle-groups.o block/nbd.o block/nbd-client.o >> block/sheepdog.o block/accounting.o block/write-threshold.o >> block/dmg.o libqemuutil.a libqemustub.a -lz -laio -lm -pthread >> -lgthread-2.0 -lglib-2.0 -lpcre -lz -lrt -lz -lcap-ng -lvdeplug >> -luuid -lgcrypt -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt >> -lgpg-error -ltasn1 -lz -lp11-kit -lutil >> c++: error: unrecognized command line option ‘-R’ >> >> (That's configure arguments '--cc=ccache gcc' '--enable-debug' >> '--static' '--disable-system', on an x86-64 Linux host.) > > You don't have a static gnutls installed, you need to add either > --disable-gnutls or --disable-tools. Shouldn't configure notice this and disable tls ? -- PMM
On 08/07/2015 17:48, Peter Maydell wrote: > On 8 July 2015 at 16:21, Paolo Bonzini <pbonzini@redhat.com> wrote: >> >> >> On 08/07/2015 16:46, Peter Maydell wrote: >>> I'm afraid this fails to link on my linux-static build: >>> >>> c++ -I/usr/include/pixman-1 -Werror -m64 -D_GNU_SOURCE >>> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes >>> -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes >>> -fno-strict-aliasing -fno-common -Wendif-labels >>> -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security >>> -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration >>> -Wold-style-definition -Wtype-limits -fstack-protector-all >>> -I/usr/include/p11-kit-1 -I/usr/include/libpng12 >>> -I/usr/include/libusb-1.0 >>> -I/home/petmay01/linaro/qemu-for-merges/tests -pthread >>> -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include >>> -g -Wl,--warn-common -m64 -static -g -o qemu-nbd qemu-nbd.o async.o >>> thread-pool.o nbd.o block.o blockjob.o main-loop.o iohandler.o >>> qemu-timer.o aio-posix.o qemu-io-cmds.o qemu-coroutine.o >>> qemu-coroutine-lock.o qemu-coroutine-io.o qemu-coroutine-sleep.o >>> coroutine-ucontext.o block/raw_bsd.o block/qcow.o block/vdi.o >>> block/vmdk.o block/cloop.o block/bochs.o block/vpc.o block/vvfat.o >>> block/qcow2.o block/qcow2-refcount.o block/qcow2-cluster.o >>> block/qcow2-snapshot.o block/qcow2-cache.o block/qed.o >>> block/qed-gencb.o block/qed-l2-cache.o block/qed-table.o >>> block/qed-cluster.o block/qed-check.o block/vhdx.o block/vhdx-endian.o >>> block/vhdx-log.o block/quorum.o block/parallels.o block/blkdebug.o >>> block/blkverify.o block/block-backend.o block/snapshot.o block/qapi.o >>> block/raw-posix.o block/linux-aio.o block/null.o block/mirror.o >>> block/io.o block/throttle-groups.o block/nbd.o block/nbd-client.o >>> block/sheepdog.o block/accounting.o block/write-threshold.o >>> block/dmg.o libqemuutil.a libqemustub.a -lz -laio -lm -pthread >>> -lgthread-2.0 -lglib-2.0 -lpcre -lz -lrt -lz -lcap-ng -lvdeplug >>> -luuid -lgcrypt -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt >>> -lgpg-error -ltasn1 -lz -lp11-kit -lutil >>> c++: error: unrecognized command line option ‘-R’ >>> >>> (That's configure arguments '--cc=ccache gcc' '--enable-debug' >>> '--static' '--disable-system', on an x86-64 Linux host.) >> >> You don't have a static gnutls installed, you need to add either >> --disable-gnutls or --disable-tools. > > Shouldn't configure notice this and disable tls ? Packaging bugs around static libraries aren't rare, and it's difficult to say who's guilty without a before/after config.log and build log. Is it okay for you to pull this and fix it during the -rc period? Also, what distro do you do this static build on? Paolo
On 8 July 2015 at 18:06, Paolo Bonzini <pbonzini@redhat.com> wrote: > On 08/07/2015 17:48, Peter Maydell wrote: >>> You don't have a static gnutls installed, you need to add either >>> --disable-gnutls or --disable-tools. >> >> Shouldn't configure notice this and disable tls ? > > Packaging bugs around static libraries aren't rare, and it's difficult > to say who's guilty without a before/after config.log and build log. Is > it okay for you to pull this and fix it during the -rc period? I guess that's probably the lesser of the two evils. > Also, what distro do you do this static build on? Ubuntu trusty. -- PMM
On 8 July 2015 at 12:19, Paolo Bonzini <pbonzini@redhat.com> wrote: > The following changes since commit f6e3035f75e5c6a73485335765ae070304c7a110: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream-smm' into staging (2015-07-06 23:37:53 +0100) > > are available in the git repository at: > > git://github.com/bonzini/qemu.git tags/for-upstream > > for you to fetch changes up to 2828a307232ffceeddec9feb6a87ac660b68b693: > > ossaudio: fix memory leak (2015-07-08 13:11:01 +0200) > > ---------------------------------------------------------------- > Bugfixes and Daniel Berrange's crypto library. > Applied, thanks. -- PMM
On 08/07/2015 22:48, Peter Maydell wrote: > On 8 July 2015 at 12:19, Paolo Bonzini <pbonzini@redhat.com> wrote: >> The following changes since commit f6e3035f75e5c6a73485335765ae070304c7a110: >> >> Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream-smm' into staging (2015-07-06 23:37:53 +0100) >> >> are available in the git repository at: >> >> git://github.com/bonzini/qemu.git tags/for-upstream >> >> for you to fetch changes up to 2828a307232ffceeddec9feb6a87ac660b68b693: >> >> ossaudio: fix memory leak (2015-07-08 13:11:01 +0200) >> >> ---------------------------------------------------------------- >> Bugfixes and Daniel Berrange's crypto library. >> > > Applied, thanks. Thank you very much! Paolo
On Wed, Jul 08, 2015 at 04:48:27PM +0100, Peter Maydell wrote: > On 8 July 2015 at 16:21, Paolo Bonzini <pbonzini@redhat.com> wrote: > > > > > > On 08/07/2015 16:46, Peter Maydell wrote: > >> I'm afraid this fails to link on my linux-static build: > >> > >> c++ -I/usr/include/pixman-1 -Werror -m64 -D_GNU_SOURCE > >> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes > >> -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes > >> -fno-strict-aliasing -fno-common -Wendif-labels > >> -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security > >> -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration > >> -Wold-style-definition -Wtype-limits -fstack-protector-all > >> -I/usr/include/p11-kit-1 -I/usr/include/libpng12 > >> -I/usr/include/libusb-1.0 > >> -I/home/petmay01/linaro/qemu-for-merges/tests -pthread > >> -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include > >> -g -Wl,--warn-common -m64 -static -g -o qemu-nbd qemu-nbd.o async.o > >> thread-pool.o nbd.o block.o blockjob.o main-loop.o iohandler.o > >> qemu-timer.o aio-posix.o qemu-io-cmds.o qemu-coroutine.o > >> qemu-coroutine-lock.o qemu-coroutine-io.o qemu-coroutine-sleep.o > >> coroutine-ucontext.o block/raw_bsd.o block/qcow.o block/vdi.o > >> block/vmdk.o block/cloop.o block/bochs.o block/vpc.o block/vvfat.o > >> block/qcow2.o block/qcow2-refcount.o block/qcow2-cluster.o > >> block/qcow2-snapshot.o block/qcow2-cache.o block/qed.o > >> block/qed-gencb.o block/qed-l2-cache.o block/qed-table.o > >> block/qed-cluster.o block/qed-check.o block/vhdx.o block/vhdx-endian.o > >> block/vhdx-log.o block/quorum.o block/parallels.o block/blkdebug.o > >> block/blkverify.o block/block-backend.o block/snapshot.o block/qapi.o > >> block/raw-posix.o block/linux-aio.o block/null.o block/mirror.o > >> block/io.o block/throttle-groups.o block/nbd.o block/nbd-client.o > >> block/sheepdog.o block/accounting.o block/write-threshold.o > >> block/dmg.o libqemuutil.a libqemustub.a -lz -laio -lm -pthread > >> -lgthread-2.0 -lglib-2.0 -lpcre -lz -lrt -lz -lcap-ng -lvdeplug > >> -luuid -lgcrypt -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt > >> -lgpg-error -ltasn1 -lz -lp11-kit -lutil > >> c++: error: unrecognized command line option ‘-R’ > >> > >> (That's configure arguments '--cc=ccache gcc' '--enable-debug' > >> '--static' '--disable-system', on an x86-64 Linux host.) > > > > You don't have a static gnutls installed, you need to add either > > --disable-gnutls or --disable-tools. > > Shouldn't configure notice this and disable tls ? Ok, I can reproduce this easily enough with those configure args on Fedora too. The original configure check would use pkg-config to get the compile/linker flags and then invoke 'compile_prog' to test them. The new configure check uses pkg-config too, but skips the 'compile_prog' test, which is why you see a difference in behaviour with these crypto patches applied. To compare before and after state though, if I go back to before my crypto patches were applied, and do ./configure --enable-debug --static --disable-system && make I get failures for many other libraries too, not least nothing seems to check for glib2 static libs being present LINK qemu-ga /usr/bin/ld: cannot find -lgthread-2.0 /usr/bin/ld: cannot find -lglib-2.0 And if I install those, I get failures for a bunch of other storage libraries too LINK qemu-nbd /usr/bin/ld: cannot find -liscsi /usr/bin/ld: cannot find -lgfapi /usr/bin/ld: cannot find -lglusterfs /usr/bin/ld: cannot find -lgfrpc /usr/bin/ld: cannot find -lgfxdr /usr/bin/ld: cannot find -lssh2 /usr/bin/ld: cannot find -lssl /usr/bin/ld: cannot find -lcrypto /usr/bin/ld: cannot find -lgssapi_krb5 /usr/bin/ld: cannot find -lkrb5 /usr/bin/ld: cannot find -lcom_err /usr/bin/ld: cannot find -lk5crypto /usr/bin/ld: cannot find -lcrypto I'm happy to fix configure to check that gnutls has working static libs, but we do have a much broader pre-existing problem in this respect. So it seems people wanting todo static builds currently have to deal with installing all the correct static bits upfront or manually disabling features in configure. Regards, Daniel
On 24/07/2015 14:52, Daniel P. Berrange wrote: > I'm happy to fix configure to check that gnutls has working static > libs, but we do have a much broader pre-existing problem in this respect. > So it seems people wanting todo static builds currently have to deal with > installing all the correct static bits upfront or manually disabling > features in configure. Ok, then it looks like a preexisting problem indeed. I'll add it to the prerequisites section of the release notes. Paolo
On 24 July 2015 at 13:53, Paolo Bonzini <pbonzini@redhat.com> wrote: > > > On 24/07/2015 14:52, Daniel P. Berrange wrote: >> I'm happy to fix configure to check that gnutls has working static >> libs, but we do have a much broader pre-existing problem in this respect. >> So it seems people wanting todo static builds currently have to deal with >> installing all the correct static bits upfront or manually disabling >> features in configure. > > Ok, then it looks like a preexisting problem indeed. Well, not entirely. This set of configure flags used to work, and now it does not. That's a regression. -- PMM
On 24/07/2015 15:00, Peter Maydell wrote: > > > I'm happy to fix configure to check that gnutls has working static > > > libs, but we do have a much broader pre-existing problem in this respect. > > > So it seems people wanting todo static builds currently have to deal with > > > installing all the correct static bits upfront or manually disabling > > > features in configure. > > > > Ok, then it looks like a preexisting problem indeed. > > Well, not entirely. This set of configure flags used to work, > and now it does not. That's a regression. Yes, it's indeed adding a new optional build prerequisite for the tools, and like all other prerequisites you need to install the static version manually. But special-casing gnutls makes little sense when the same problem is present for all other libraries. Paolo
On 24 July 2015 at 14:00, Peter Maydell <peter.maydell@linaro.org> wrote: > On 24 July 2015 at 13:53, Paolo Bonzini <pbonzini@redhat.com> wrote: >> Ok, then it looks like a preexisting problem indeed. > > Well, not entirely. This set of configure flags used to work, > and now it does not. That's a regression. In particular, configure --static --disable-tools --disable-system now fails, and I think that's a perfectly sensible set of options. It fails trying to link LINK tests/qemu-iotests/socket_scm_helper: ...why are we trying to build a helper program in tests/ for a simple 'make' (not 'make check') ? Why are we building it in a config which has disabled building tools and system exes? Probably avoiding building this random exe would fix things. thanks -- PMM
On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: > On 24 July 2015 at 14:00, Peter Maydell <peter.maydell@linaro.org> wrote: > > On 24 July 2015 at 13:53, Paolo Bonzini <pbonzini@redhat.com> wrote: > >> Ok, then it looks like a preexisting problem indeed. > > > > Well, not entirely. This set of configure flags used to work, > > and now it does not. That's a regression. > > In particular, configure --static --disable-tools --disable-system > now fails, and I think that's a perfectly sensible set of options. Presumably you have previously figured out which static libs you needed to install on your build machine previously though ? As that command line fails to verify presence of any static libraries at all - configure doesn't stop and tell me to install any static libraries at all on my machine which has no static libs present - it just fails to build later > > It fails trying to link > LINK tests/qemu-iotests/socket_scm_helper: > > ...why are we trying to build a helper program in tests/ for > a simple 'make' (not 'make check') ? Why are we building it > in a config which has disabled building tools and system exes? > > Probably avoiding building this random exe would fix things. Regards, Daniel
On 24 July 2015 at 14:10, Daniel P. Berrange <berrange@redhat.com> wrote: > On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: >> On 24 July 2015 at 14:00, Peter Maydell <peter.maydell@linaro.org> wrote: >> > On 24 July 2015 at 13:53, Paolo Bonzini <pbonzini@redhat.com> wrote: >> >> Ok, then it looks like a preexisting problem indeed. >> > >> > Well, not entirely. This set of configure flags used to work, >> > and now it does not. That's a regression. >> >> In particular, configure --static --disable-tools --disable-system >> now fails, and I think that's a perfectly sensible set of options. > > Presumably you have previously figured out which static libs you > needed to install on your build machine previously though ? > As that command line fails to verify presence of any static > libraries at all - configure doesn't stop and tell me to install > any static libraries at all on my machine which has no static > libs present - it just fails to build later Yes, but the set of required static libs for linux-user is pretty tiny, and it doesn't generally change. In particular since we don't want to link gnutls into linux-user we shouldn't be requiring the user to scrape up a static lib of it to keep things working. Requiring gnutls at all for these configure options is just a bug (in configure or the makefile). -- PMM
On Fri, Jul 24, 2015 at 02:31:12PM +0100, Peter Maydell wrote: > On 24 July 2015 at 14:10, Daniel P. Berrange <berrange@redhat.com> wrote: > > On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: > >> On 24 July 2015 at 14:00, Peter Maydell <peter.maydell@linaro.org> wrote: > >> > On 24 July 2015 at 13:53, Paolo Bonzini <pbonzini@redhat.com> wrote: > >> >> Ok, then it looks like a preexisting problem indeed. > >> > > >> > Well, not entirely. This set of configure flags used to work, > >> > and now it does not. That's a regression. > >> > >> In particular, configure --static --disable-tools --disable-system > >> now fails, and I think that's a perfectly sensible set of options. > > > > Presumably you have previously figured out which static libs you > > needed to install on your build machine previously though ? > > As that command line fails to verify presence of any static > > libraries at all - configure doesn't stop and tell me to install > > any static libraries at all on my machine which has no static > > libs present - it just fails to build later > > Yes, but the set of required static libs for linux-user is pretty > tiny, and it doesn't generally change. In particular since we > don't want to link gnutls into linux-user we shouldn't be requiring > the user to scrape up a static lib of it to keep things working. > Requiring gnutls at all for these configure options is just a bug > (in configure or the makefile). Ok, I'll send a patch to auto-disable it when missing Regards, Daniel
On 24/07/2015 15:48, Daniel P. Berrange wrote: > > Yes, but the set of required static libs for linux-user is pretty > > tiny, and it doesn't generally change. In particular since we > > don't want to link gnutls into linux-user we shouldn't be requiring > > the user to scrape up a static lib of it to keep things working. > > Requiring gnutls at all for these configure options is just a bug > > (in configure or the makefile). > > Ok, I'll send a patch to auto-disable it when missing If the problem is just socket-scm-helper, which is only used to test system emulators, we can make that depend on CONFIG_SOFTMMU. Paolo
diff --git a/block/quorum.c b/block/quorum.c index 5938864..4e66221 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -24,8 +24,6 @@ #include "qapi-event.h" #include "crypto/hash.h" -#include <glib/gi18n.h> - #define HASH_LENGTH 32 #define QUORUM_OPT_VOTE_THRESHOLD "vote-threshold" diff --git a/crypto/cipher.c b/crypto/cipher.c index 663bbe3..024a00c 100644 --- a/crypto/cipher.c +++ b/crypto/cipher.c @@ -33,8 +33,7 @@ qcrypto_cipher_validate_key_length(QCryptoCipherAlgorithm alg, size_t nkey, Error **errp) { - if (alg < 0 || - alg >= QCRYPTO_CIPHER_ALG_LAST) { + if ((unsigned)alg >= QCRYPTO_CIPHER_ALG_LAST) { error_setg(errp, "Cipher algorithm %d out of range", alg); return false;