Re: [Qemu-devel] [PATCH v1 2/5] configure: ensure ldflags propagated to config_host

2016-01-29 Thread Alex Bennée
Paolo Bonzini writes: > On 28/01/2016 11:15, Alex Bennée wrote: >> diff --git a/configure b/configure >> index bd29ba7..148b79a 100755 >> --- a/configure >> +++ b/configure >> @@ -5871,7 +5871,7 @@ if test "$target_linux_user" = "yes" -o >> "$target_bsd_user" = "yes" ;

Re: [Qemu-devel] [PATCH v1 2/5] configure: ensure ldflags propagated to config_host

2016-01-28 Thread Paolo Bonzini
On 28/01/2016 11:15, Alex Bennée wrote: > index d0de2d4..d30532f 100644 > --- a/Makefile > +++ b/Makefile > @@ -329,9 +329,9 @@ qemu-ga: qemu-ga$(EXESUF) $(QGA_VSS_PROVIDER) > $(QEMU_GA_MSI) > endif > > ivshmem-client$(EXESUF): $(ivshmem-client-obj-y) > - $(call LINK, $^) > + $(call

[Qemu-devel] [PATCH v1 2/5] configure: ensure ldflags propagated to config_host

2016-01-28 Thread Alex Bennée
If for example you want to use the thread sanitizer you want to ensure all binaries have the extra linker flags applied: ./configure ${TARGETS} --cc=gcc-5 --cxx=g++-5 \ --extra-cflags="-fsanitize=thread" --extra-ldflags="-fsanitize=thread" Or possibly: ./configure ${TARGETS}

Re: [Qemu-devel] [PATCH v1 2/5] configure: ensure ldflags propagated to config_host

2016-01-28 Thread Paolo Bonzini
On 28/01/2016 11:15, Alex Bennée wrote: > diff --git a/configure b/configure > index bd29ba7..148b79a 100755 > --- a/configure > +++ b/configure > @@ -5871,7 +5871,7 @@ if test "$target_linux_user" = "yes" -o > "$target_bsd_user" = "yes" ; then >ldflags="$ldflags $textseg_ldflags" > fi >