[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