Re: [Qemu-devel] [RFC PATCH] capstone: fix building using system package

[miqlas]

Hi,

afaik (but not tested) pkgconfig --cflags reports /includes on linux, 
and it does the same on Haiku too.
I'm not against to change our capstone recipe, but please, if you can 
check it on Linux and report it back, as i don't want to break other 
software.
Thanks for the nice talk, guys!

--miqlas


2018-02-15 19:39 keltezéssel, Philippe Mathieu-Daudé írta:
> Hi Sergei,
>
> On 02/15/2018 03:21 PM, Sergei Trofimovich wrote:
> > On Thu, 15 Feb 2018 14:35:39 -0300
> > Philippe Mathieu-Daudé <address@hidden> wrote:
> >
> > >   #else
> > > +#include <capstone/capstone.h>
> > I think it's incorrect. 'pkg-config' already reports 'capstone/' path:
> >      $ pkg-config --cflags capstone
> >      -I/usr/include/capstone
> Glad to hear feedback from a Gentoo developer!
>
> Ok so the problem Haiku only, which we don't support anymore.
>
> >      $ ls /usr/include/capstone/capstone.h
> >      /usr/include/capstone/capstone.h
> >
> > Thus I would guess
> >      #include <capstone.h>
> > is still correct for system include path as well (contradicts the example).
> My guess is the example is probabilisticly safer for people compiling
> without using 'pkg-config'.
>
> > qemu just needs to use 'pkg-config' to discover the include path and
> > libs. Maybe new capstone release has different pkgconfig setup?
> I think it is safer to drop this patch.
>
> Thanks for your review!
>
> Phil.