[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 02/16] ci: fixes msys2 build by upgrading capstone to 4.0.2
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PULL 02/16] ci: fixes msys2 build by upgrading capstone to 4.0.2 |
Date: |
Tue, 8 Sep 2020 21:07:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/8/20 8:59 PM, Eric Blake wrote:
> On 9/8/20 1:49 PM, Yonggang Luo wrote:
>> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
>> ---
>> capstone | 2 +-
>> configure | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> The commit message is sparse; it might be nice to give more details
> about what error is fixed, or possibly even mention of the fact of which
> capstone commit id that is in 4.0.2 but not the current version matters
> to qemu on msys.
>
>>
>> diff --git a/capstone b/capstone
>> index 22ead3e0bf..1d23053284 160000
>> --- a/capstone
>> +++ b/capstone
>> @@ -1 +1 @@
>> -Subproject commit 22ead3e0bfdb87516656453336160e0a37b066bf
>> +Subproject commit 1d230532840a37ac032c6ab80128238fc930c6c1
>> diff --git a/configure b/configure
>> index 4231d56bcc..f4f8bc3756 100755
>> --- a/configure
>> +++ b/configure
>> @@ -5156,7 +5156,7 @@ case "$capstone" in
>> LIBCAPSTONE=libcapstone.a
>> fi
>> capstone_libs="-Lcapstone -lcapstone"
>> - capstone_cflags="-I${source_path}/capstone/include"
>> + capstone_cflags="-I${source_path}/capstone/include
>> -I${source_path}/capstone/include/capstone"
>
> This change was not mentioned in the commit message. Did capstone 4.0.2
> change where its include files live? Or is it a separate bug, and you
> are fixing two things at once (in which case, doing two separate commits
> might be nicer)?
As Richard Henderson introduced the capstone submodule, it would be
nice to have his Ack-by before updating it.
Thanks,
Phil.
- [PULL 00/16] Msys2 patches patches, Yonggang Luo, 2020/09/08
- [PULL 01/16] block: Fixes nfs on msys2/mingw, Yonggang Luo, 2020/09/08
- [PULL 02/16] ci: fixes msys2 build by upgrading capstone to 4.0.2, Yonggang Luo, 2020/09/08
- [PULL 03/16] configure: Fixes ncursesw detection under msys2/mingw and enable curses, Yonggang Luo, 2020/09/08
- [PULL 04/16] curses: Fixes curses compiling errors., Yonggang Luo, 2020/09/08
- [PULL 05/16] tests: disable /char/stdio/* tests in test-char.c on win32, Yonggang Luo, 2020/09/08
- [PULL 06/16] ci: Enable msys2 ci in cirrus, Yonggang Luo, 2020/09/08
- [PULL 07/16] tests: Trying fixes test-replication.c on msys2/mingw., Yonggang Luo, 2020/09/08
- [PULL 08/16] block: get file-win32.c handle locking option consistence with file-posix.c, Yonggang Luo, 2020/09/08
- [PULL 09/16] osdep: These function are only available on Non-Win32 system., Yonggang Luo, 2020/09/08