|
From: | Eric Blake |
Subject: | Re: [PULL 02/16] ci: fixes msys2 build by upgrading capstone to 4.0.2 |
Date: | Tue, 8 Sep 2020 13:59:53 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
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)?
-- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |