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

Aurelien Jacobs aurel
Sun Jul 6 22:53:00 CEST 2008


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.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: uninstalled-pc-dir.diff
Type: text/x-patch
Size: 373 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080706/adec4972/attachment.bin>



More information about the ffmpeg-devel mailing list