r1564 - in /experimental/ffmpeg-debian/debian: changelog patches/050_fix_pkgconfig_files.patch

[siretart at users.alioth.debian.org]

Author: siretart
Date: Mon Nov 10 20:39:58 2008
New Revision: 1564

URL: http://svn.debian.org/wsvn/pkg-multimedia/?sc=1&rev=1564
Log:
Adjust pkg-files to no longer put unnecessary dependencies in the generated
.pc files. Closes: #504220

Added:
    experimental/ffmpeg-debian/debian/patches/050_fix_pkgconfig_files.patch
Modified:
    experimental/ffmpeg-debian/debian/changelog

Modified: experimental/ffmpeg-debian/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-multimedia/experimental/ffmpeg-debian/debian/changelog?rev=1564&op=diff
==============================================================================
--- experimental/ffmpeg-debian/debian/changelog (original)
+++ experimental/ffmpeg-debian/debian/changelog Mon Nov 10 20:39:58 2008
@@ -1,8 +1,10 @@
 ffmpeg-debian (3:0.svn20081108-2) experimental; urgency=low
 
   * bump standards version to version 3.8.0, no changes needed
-
- -- Reinhard Tartler <siretart at tauware.de>  Mon, 10 Nov 2008 21:35:22 +0100
+  * Adjust pkg-files to no longer put unnecessary dependencies in the generated
+    .pc files. Closes: #504220
+
+ -- Reinhard Tartler <siretart at tauware.de>  Mon, 10 Nov 2008 21:37:16 +0100
 
 ffmpeg-debian (3:0.svn20081108-1) experimental; urgency=low
 

Added: experimental/ffmpeg-debian/debian/patches/050_fix_pkgconfig_files.patch
URL: http://svn.debian.org/wsvn/pkg-multimedia/experimental/ffmpeg-debian/debian/patches/050_fix_pkgconfig_files.patch?rev=1564&op=file
==============================================================================
--- experimental/ffmpeg-debian/debian/patches/050_fix_pkgconfig_files.patch (added)
+++ experimental/ffmpeg-debian/debian/patches/050_fix_pkgconfig_files.patch Mon Nov 10 20:39:58 2008
@@ -1,0 +1,29 @@
+patch by Reinhard Tartler <siretart at debian.org>
+
+avoid using Requires.private. According to pkg-config upstream they are only
+necessary if a library exposes internal types (structs, enums) of the
+"required" libraries. This is not the case for ffmpeg, so Libs.private should
+be used instead. Moreover, Requires alone is pretty pointless.
+
+Discussion about this started at:
+
+http://permalink.gmane.org/gmane.comp.video.ffmpeg.devel/76897
+http://comments.gmane.org/gmane.comp.video.ffmpeg.devel/76897
+
+--- a/configure
++++ b/configure
+@@ -2357,11 +2357,11 @@ includedir=$incdir
+ Name: $name
+ Description: $comment
+ Version: $version
+-Requires: $(disabled shared && echo $requires)
+-Requires.private: $(enabled shared && echo $requires)
++Requires:
++Requires.private:
+ Conflicts:
+ Libs: -L\${libdir} -l${shortname} $(disabled shared && echo $libs)
+-Libs.private: $(enabled shared && echo $libs)
++Libs.private: $(enabled shared && echo $libs $requires)
+ Cflags: -I\${includedir}
+ EOF
+ cat <<EOF > $name/$name-uninstalled.pc