[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