[Buildroot] [PATCH 1/1] ti-gfx: pkg-config should also set parent include dir

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri May 30 20:27:18 UTC 2014


Dear Spenser Gilliland,

On Fri, 30 May 2014 14:57:19 -0500, Spenser Gilliland wrote:

> > Yes, Qt5's cmake files assume /usr/include not /usr/include/GLES2.
> >
> > I added -I${includedir} instead of removing only "GLES2" because then
> nothing will break,
> > and the other .pc file also sets both dirs.
> > But it seems common practice is to not add the uppercase dir names in the
> pkg-congig file.

You should fix your e-mail client, as it very badly handles quotations.

> This patch won't break anything.  But it would be nice if you could go back
> and reverify this just to be sure.
> 
> My reasoning is that,  /usr/include is a default include dir which all
> compilers should use automatically.   There should not be a need to
> manually specify it.

Agreed, and therefore I'd like to understand in more details why the
patch is needed. We typically don't commit things we don't understand
why they are needed :)

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list