[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] build-sys: do not include Windows SLIRP dependencies in $LIB
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH] build-sys: do not include Windows SLIRP dependencies in $LIBS |
Date: |
Wed, 11 Dec 2019 16:06:50 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 |
On 11/12/19 15:37, Samuel Thibault wrote:
> Paolo Bonzini, le mer. 11 déc. 2019 15:23:23 +0100, a ecrit:
>> When including the internal SLIRP library, we should add all the libraries
>> that
>> it needs for the build. Right now they are all included by QEMU, but
>> -liphlpapi
>> is not needed without slirp. Move it from LIBS to slirp_libs.
>>
>> Based on a patch by Marc-André Lureau.
>>
>> Signed-off-by: Paolo Bonzini <address@hidden>
>
> Acked-by: Samuel Thibault <address@hidden>
Ok, I'll queue it. But you could use "make docker-test-mingw@fedora". :)
Paolo
> (I don't have a win environment to test this)
>
>> ---
>> configure | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 6099be1..d16dad2 100755
>> --- a/configure
>> +++ b/configure
>> @@ -926,7 +926,7 @@ if test "$mingw32" = "yes" ; then
>> DSOSUF=".dll"
>> # MinGW needs -mthreads for TLS and macro _MT.
>> QEMU_CFLAGS="-mthreads $QEMU_CFLAGS"
>> - LIBS="-lwinmm -lws2_32 -liphlpapi $LIBS"
>> + LIBS="-lwinmm -lws2_32 $LIBS"
>> write_c_skeleton;
>> if compile_prog "" "-liberty" ; then
>> LIBS="-liberty $LIBS"
>> @@ -6069,6 +6069,9 @@ case "$slirp" in
>> mkdir -p slirp
>> slirp_cflags="-I\$(SRC_PATH)/slirp/src -I\$(BUILD_DIR)/slirp/src"
>> slirp_libs="-L\$(BUILD_DIR)/slirp -lslirp"
>> + if test "$mingw32" = "yes" ; then
>> + slirp_libs="$slirp_libs -lws2_32 -liphlpapi"
>> + fi
>> ;;
>>
>> system)
>> --
>> 1.8.3.1
>>
>>
>