On Mon, May 7, 2012 at 7:19 PM, Carsten Haitzler <ras...@rasterman.com> wrote: > On Sun, 6 May 2012 23:42:52 +0200 Vincent Torri <vincent.to...@gmail.com> > said: > >> On Sun, May 6, 2012 at 10:26 PM, Paul Cercueil <p...@crapouillou.net> wrote: >> > >> > Hello, >> > >> > I've been trying to integrate the EFL libraries inside our small >> > distribution built using Buildroot. However, I am facing a nasty >> > problem: the installed .pc files (that are used by pkg-config) do >> > contain incorrect paths. >> > >> > For instance, evas.pc will contain "-I${includedir}/evas-1 >> > -I/opt/toolchain/usr/include/eina-1". As a result, the ./configure >> > script for (ie.) Edje will prepend its --prefix to those paths, which >> > will become "-I/opt/toolchain/usr/include/evas-1 >> > -I/opt/toolchain/opt/toolchain/usr/include/eina-1". >> > >> > By looking at the source code, it appears that $EINA_CFLAGS is >> > referenced on evas.pc.in. That value is initialized by pkg-config on the >> > ./configure script. Usually, using $EINA_CFLAGS on eina.pc.in should be >> > fine; but Buildroot sets the PKG_CONFIG_SYSROOT_DIR environment >> > variable, which is prepended to the paths returned by pkg-config. I >> > believe that environment variable should be cleared before pkg-config is >> > called. >> >> I'm really wondering why EINA_CFLAGS appears in evas.pc.in. It should >> not. The Cflags value in evas.pc.in should contain the path needed to > > it should. Evas.h includes Eina.h. someone has to be able to just use the evas > pkgconfig info to compile something that JUST includes (and links to) evas. > thus to make it work it MUST also add in the CFLAGS for eina. the thing > linking > to evas should need to not know its dependency chain - if it changes in future > or not.
Eina.h includes Evil.h and i have absolutely no need to badly hack eina.pc like that... Hopefully pkg-config does everything it needs to append the path. If there is a bug, it must be fixed properly, and not badly like you did For your information: http://people.freedesktop.org/~dbn/pkg-config-guide.html "Cflags: The compiler flags specific to this package and any required libraries that don't support pkg-config. If the required libraries support pkg-config, they should be added to Requires or Requires.private." Vincent ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel