Re: Qemu-devel Digest, Vol 198, Issue 358

Re: Qemu-devel Digest, Vol 198, Issue 358

[G 3]

[-- Attachment #1: Type: text/plain, Size: 2348 bytes --]

On Tue, Sep 24, 2019 at 1:05 PM G 3 <programmingkidx@gmail.com> wrote:

> On top
>
>
> Message: 9
>> Date: Tue, 24 Sep 2019 10:56:53 -0000
>> From: Adrian Vladu <avladu@cloudbasesolutions.com>
>> To: qemu-devel@nongnu.org
>> Subject: [Bug 1826175] Re: Compilation on MSYS2/MinGW-w64 fails with
>>         error: "No rule to make target capstone.lib"
>> Message-ID:
>>         <
>> 156932261321.27303.8940310087786694778.malone@soybean.canonical.com>
>> Content-Type: text/plain; charset="utf-8"
>>
>> The fix in upstream capstone has been merged:
>>
>> https://github.com/aquynh/capstone/commit/29893c63e34ee21846744d02c396ae3c801b936b
>>
>> --
>> You received this bug notification because you are a member of qemu-
>> devel-ml, which is subscribed to QEMU.
>> https://bugs.launchpad.net/bugs/1826175
>>
>> Title:
>>   Compilation on MSYS2/MinGW-w64 fails with error: "No rule to make
>>   target capstone.lib"
>>
>> Status in QEMU:
>>   New
>>
>> Bug description:
>>   I submitted this bug to Capstone directly but I figured it'd be useful
>>   to post it here too. The IS_MINGW check in the Makefile for Capstone
>>   fails under MSYS2 MinGW-w64 because cc --version doesn't have mingw in
>>   the output anymore:
>>
>>   $ whereis cc
>>   cc: /mingw64/bin/cc.exe
>>
>>   $ cc --version
>>   cc.exe (Rev2, Built by MSYS2 project) 8.3.0
>>   Copyright (C) 2018 Free Software Foundation, Inc.
>>   This is free software; see the source for copying conditions.  There is
>> NO
>>   warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
>> PURPOSE.
>>
>>   Really simple patch:
>>
>>   diff --git "a/Makefile" "b/Makefile"
>>   index 063f50db..1d9f042e 100644
>>   --- "a/Makefile"
>>   +++ "b/Makefile"
>>   @@ -288,7 +288,7 @@ CFLAGS := $(CFLAGS:-fPIC=)
>>    # On Windows we need the shared library to be executable
>>    else
>>    # mingw?
>>   -IS_MINGW := $(shell $(CC) --version | grep -i mingw | wc -l)
>>   +IS_MINGW := $(shell $(CC) --version | grep -i msys2 | wc -l)
>>    ifeq ($(IS_MINGW),1)
>>    EXT = dll
>>    AR_EXT = lib
>>
>> To manage notifications about this bug go to:
>> https://bugs.launchpad.net/qemu/+bug/1826175/+subscriptions
>>
>> ********************************************
>>
>
Hi Peter, could we make capstone disabled by default please? It does cause
compiling problems and isn't needed to use QEMU.

[-- Attachment #2: Type: text/html, Size: 3699 bytes --]