On Sun, Nov 13, 2022 at 11:09:42PM +0300, Michael Tokarev wrote:
> The upcoming capstone 5.0 drops support for the old way
> of including its header, due to this change:
> https://github.com/capstone-engine/capstone/commit/6656bcb63ab4e87dc6079bd6b6b12cc8dd9b2ad8
> The official way is to use <capstone/capstone.h>
>
> This change has already been proposed before, see
> https://patchwork.kernel.org/project/qemu-devel/patch/20180215173539.11033-1-f4...@amsat.org/
> but it didn't find its way into qemu at that time.
>
> On current systems, using <capstone/capstone.h> works
> now (despite the pkg-config-supplied -I/usr/include/capstone) -
> since on all systems capstone headers are put into capstone/
> subdirectory of a system include dir. So this change is
> compatible with both the obsolete way of including it
> and the only future way.
AFAIR, macOS HomeBrew does not put anything into the system
include dir, and always requires -I flags to be correct.
> diff --git a/meson.build b/meson.build
> index cf3e517e56..6f34c963f7 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -2680,12 +2680,7 @@ if not get_option('capstone').auto() or have_system or
> have_user
> capstone = dependency('capstone', version: '>=3.0.5',
> kwargs: static_kwargs, method: 'pkg-config',
> required: get_option('capstone'))
> -
> - # Some versions of capstone have broken pkg-config file
> - # that reports a wrong -I path, causing the #include to
> - # fail later. If the system has such a broken version
> - # do not use it.
> - if capstone.found() and not cc.compiles('#include <capstone.h>',
> + if capstone.found() and not cc.compiles('#include <capstone/capstone.h>',
> dependencies: [capstone])
To retain back compat this could probe for both ways
if capstone.found()
if cc.compiles('#include <capstone/capstone.h>',
dependencies: [capstone])
...
else if cc.compiles('#include <capstone.h>',
dependencies: [capstone])
...
then, the source file can try the correct #include based on what
we detect works here.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
