Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Peter Maydell
On 24 July 2015 at 15:25, Peter Maydell wrote: > (b) make sure configure doesn't put the results of > 'pkg-config --libs --static gnutls' into compiler commandlines > (it is: > -R/usr/lib/x86_64-linux-gnu -lgnutls -lgcrypt -lgpg-error -ltasn1 -lz > -lp11-kit FWIW, I've filed an Ubuntu bug: https

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Peter Maydell
On 24 July 2015 at 14:55, Paolo Bonzini wrote: > > > On 24/07/2015 15:31, Peter Maydell 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 requirin

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
On 24/07/2015 16:19, Daniel P. Berrange wrote: >> > Right... We need to introduce CONFIG_CRYPTO and use it in >> > tests/Makefile too. > Except it was a goal of this work that we do not have such config > parameters leak outside to callers. CONFIG_CRYPTO would disable all of crypto/ if neither

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Daniel P. Berrange
On Fri, Jul 24, 2015 at 04:15:31PM +0200, Paolo Bonzini wrote: > > > On 24/07/2015 16:14, Daniel P. Berrange wrote: > >> > diff --git a/Makefile.objs b/Makefile.objs > >> > index f094eff..9d988d9 100644 > >> > --- a/Makefile.objs > >> > +++ b/Makefile.objs > >> > @@ -2,7 +2,9 @@ > >> > # Common

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
On 24/07/2015 16:14, Daniel P. Berrange wrote: >> > diff --git a/Makefile.objs b/Makefile.objs >> > index f094eff..9d988d9 100644 >> > --- a/Makefile.objs >> > +++ b/Makefile.objs >> > @@ -2,7 +2,9 @@ >> > # Common libraries for tools and emulators >> > stub-obj-y = stubs/ >> > util-obj-y = ut

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Daniel P. Berrange
On Fri, Jul 24, 2015 at 03:55:56PM +0200, Paolo Bonzini wrote: > > > On 24/07/2015 15:31, Peter Maydell 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 s

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
On 24/07/2015 15:31, Peter Maydell 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 th

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
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

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Daniel P. Berrange
On Fri, Jul 24, 2015 at 02:31:12PM +0100, Peter Maydell wrote: > On 24 July 2015 at 14:10, Daniel P. Berrange wrote: > > On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: > >> On 24 July 2015 at 14:00, Peter Maydell wrote: > >> > On 24 July 2015 at 13:53, Paolo Bonzini wrote: > >> >

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Peter Maydell
On 24 July 2015 at 14:10, Daniel P. Berrange wrote: > On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: >> On 24 July 2015 at 14:00, Peter Maydell wrote: >> > On 24 July 2015 at 13:53, Paolo Bonzini wrote: >> >> Ok, then it looks like a preexisting problem indeed. >> > >> > Well, no

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Daniel P. Berrange
On Fri, Jul 24, 2015 at 02:06:44PM +0100, Peter Maydell wrote: > On 24 July 2015 at 14:00, Peter Maydell wrote: > > On 24 July 2015 at 13:53, Paolo Bonzini wrote: > >> Ok, then it looks like a preexisting problem indeed. > > > > Well, not entirely. This set of configure flags used to work, > > an

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Peter Maydell
On 24 July 2015 at 14:00, Peter Maydell wrote: > On 24 July 2015 at 13:53, Paolo Bonzini 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

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
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 t

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Peter Maydell
On 24 July 2015 at 13:53, Paolo Bonzini 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 cu

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Paolo Bonzini
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

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-24 Thread Daniel P. Berrange
On Wed, Jul 08, 2015 at 04:48:27PM +0100, Peter Maydell wrote: > On 8 July 2015 at 16:21, Paolo Bonzini 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 > >>

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-09 Thread Paolo Bonzini
On 08/07/2015 22:48, Peter Maydell wrote: > On 8 July 2015 at 12:19, Paolo Bonzini 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) >> >

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Peter Maydell
On 8 July 2015 at 12:19, Paolo Bonzini 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:

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Peter Maydell
On 8 July 2015 at 18:06, Paolo Bonzini 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

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Paolo Bonzini
On 08/07/2015 17:48, Peter Maydell wrote: > On 8 July 2015 at 16:21, Paolo Bonzini 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

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Peter Maydell
On 8 July 2015 at 16:21, Paolo Bonzini 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-

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Paolo Bonzini
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-prototype

Re: [Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Peter Maydell
On 8 July 2015 at 12:19, Paolo Bonzini 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:

[Qemu-devel] [PULL v3 00/12] Final changes for 2.4-rc0

2015-07-08 Thread Paolo Bonzini
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