[Ffmpeg-devel] [PATCH] install libav(codec|format).pc/libpostproc.pc
Aurelien Jacobs
aurel
Tue May 17 23:21:02 CEST 2005
On Wed, 11 May 2005 12:43:19 +0200
j at v2v.cc wrote:
> attached patch creates pkg-config .pc files that reflect
> the configuration of libavcodec/libavformat and libpostproc.
> [snip]
>
> +#libavcodec.pc
> +echo 'prefix='$prefix'
> +exec_prefix=${prefix}
> [blabla]
> +Libs: -L${libdir} -lavcodec '$extralibs'
> +Cflags: -I${includedir} -I${includedir}/ffmpeg
> +' > libavcodec.pc
This way of using quotes to surround the echo string which
contains quotes itself, sounds a bit strange to me.
I would rather see a here-document such as :
cat <<EOF >libavcodec.pc
prefix='$prefix'
[blabla]
Cflags: -I${includedir} -I${includedir}/ffmpeg
EOF
> +#libavformat.pc
> +echo 'prefix='$prefix'
> +exec_prefix=${prefix}
> +libdir=${exec_prefix}/lib
> +includedir=${prefix}/include
> +
> +Name: libavformat
> +Description: This is the container format library of the ffmpeg project.
> +Version: '$version'
> +Requires: '$requires'
Here I guess that requires should also contain "libavcodec = $version" as
libavformat is linked to libavcodec
> +echo 'prefix=
> +exec_prefix=
> +libdir=${pcfiledir}/libavformat
> +includedir=${pcfiledir}/libavformat
> +
> +Name: libavcodec
wrong copy/paste... should be libavformat here.
> +Description: This is the codec library of the ffmpeg project.
> +Version: '$version'
> +Requires: '$requires'
> +Conflicts:
> +Libs: ${libdir}/'$LIBPREF'avformat'$LIBSUF' '$extralibs'
> +Cflags: -I${includedir}
> +' > libavformat-uninstalled.pc
Except those small remarks, those .pc files seems good and it would be
nice to get this included.
Aurel
More information about the ffmpeg-devel
mailing list