[FFmpeg-devel] [PATCH] create all pkg-config files in their own subdirectory

Diego Biurrun diego
Mon Jul 7 13:49:47 CEST 2008


On Sun, Jul 06, 2008 at 10:53:00PM +0200, Aurelien Jacobs wrote:
> M?ns Rullg?rd wrote:
> 
> > Jack Jansen <Jack.Jansen at cwi.nl> writes:
> > 
> > > On  28-Jun-2008, at 03:09 , M?ns Rullg?rd wrote:
> > >
> > >>> -libdir=\${pcfiledir}/$name
> > >>> +libdir=\${pcfiledir}
> > >>
> > >> If that is right, ...
> > >>
> > >>> includedir=\${pcfiledir}
> > >>
> > >> then that is wrong.
> > >
> > > No: the pattern seems to be to do
> > >
> > > 	#include "libavcodec/avcodec.h"
> > >
> > > currently. That was different in the past, IIRC, but nowadays this is
> > > what most ffmpeg code uses.
> > 
> > Yes, that is how #include directives should look.  Assuming
> > ${pcfiledir} is the directory containing the .pc file, supplying
> > -I${pcfiledir} is wrong when the .pc file is in the same directory as
> > headers/libs.
> 
> True.
> Attached patch should fix this issue properly.
> I will apply it in a few days if no one cares.
> 
> --- configure	(revision 14095)
> +++ configure	(working copy)
> @@ -2212,8 +2212,8 @@
>  cat <<EOF > $name/$name-uninstalled.pc
>  prefix=
>  exec_prefix=
> -libdir=\${pcfiledir}/$name
> -includedir=\${pcfiledir}
> +libdir=\${pcfiledir}
> +includedir=\${pcfiledir}/..

This looks wrong to me.  I don't see the problem in our current
pkg-config files...

Diego




More information about the ffmpeg-devel mailing list