[libvpx/f16] fixup pc file handling

Tom Callaway spot at fedoraproject.org
Tue May 29 14:52:14 UTC 2012


commit 93d7c8c4a54bba320467851f8d6fae05318d9188
Author: Tom Callaway <spot at fedoraproject.org>
Date:   Tue May 29 10:52:10 2012 -0400

    fixup pc file handling

 libvpx-v1.0.0-pcfix.patch |   21 +++++++++++++++++++++
 libvpx.spec               |   23 ++++++++++-------------
 2 files changed, 31 insertions(+), 13 deletions(-)
---
diff --git a/libvpx-v1.0.0-pcfix.patch b/libvpx-v1.0.0-pcfix.patch
new file mode 100644
index 0000000..6691620
--- /dev/null
+++ b/libvpx-v1.0.0-pcfix.patch
@@ -0,0 +1,21 @@
+diff -up libvpx-v1.0.0/libs.mk.pcfix libvpx-v1.0.0/libs.mk
+--- libvpx-v1.0.0/libs.mk.pcfix	2012-01-27 13:36:39.000000000 -0500
++++ libvpx-v1.0.0/libs.mk	2012-05-29 10:48:27.348425023 -0400
+@@ -232,7 +232,7 @@ vpx.pc: config.mk libs.mk
+ 	$(qexec)echo '# pkg-config file from libvpx $(VERSION_STRING)' > $@
+ 	$(qexec)echo 'prefix=$(PREFIX)' >> $@
+ 	$(qexec)echo 'exec_prefix=$${prefix}' >> $@
+-	$(qexec)echo 'libdir=$${prefix}/lib' >> $@
++	$(qexec)echo 'libdir=$${prefix}/$(LIBSUBDIR)' >> $@
+ 	$(qexec)echo 'includedir=$${prefix}/include' >> $@
+ 	$(qexec)echo '' >> $@
+ 	$(qexec)echo 'Name: vpx' >> $@
+@@ -240,7 +240,7 @@ vpx.pc: config.mk libs.mk
+ 	$(qexec)echo 'Version: $(VERSION_MAJOR).$(VERSION_MINOR).$(VERSION_PATCH)' >> $@
+ 	$(qexec)echo 'Requires:' >> $@
+ 	$(qexec)echo 'Conflicts:' >> $@
+-	$(qexec)echo 'Libs: -L$${libdir} -lvpx' >> $@
++	$(qexec)echo 'Libs: -L$${libdir} -lvpx -lm' >> $@
+ 	$(qexec)echo 'Cflags: -I$${includedir}' >> $@
+ INSTALL-LIBS-yes += $(LIBSUBDIR)/pkgconfig/vpx.pc
+ INSTALL_MAPS += $(LIBSUBDIR)/pkgconfig/%.pc %.pc
diff --git a/libvpx.spec b/libvpx.spec
index 6215fb4..d803cb9 100644
--- a/libvpx.spec
+++ b/libvpx.spec
@@ -2,15 +2,13 @@ Name:			libvpx
 Summary:		VP8 Video Codec SDK
 Version:		1.0.0
 %global soversion	%{version}
-Release:		1%{?dist}
+Release:		3%{?dist}
 License:		BSD
 Group:			System Environment/Libraries
 Source0:		http://webm.googlecode.com/files/%{name}-v%{version}.tar.bz2
-# Probably this should be dropped now that upstream ships a vpx.pc;
-# not for F16 though
-Source1:		libvpx.pc
 # Thanks to debian.
 Source2:		libvpx.ver
+Patch0:			libvpx-v1.0.0-pcfix.patch
 URL:			http://www.webmproject.org/tools/vp8-sdk/
 %ifarch %{ix86} x86_64
 BuildRequires:		yasm
@@ -42,6 +40,7 @@ and decoder.
 
 %prep
 %setup -q -n %{name}-v%{version}
+%patch0 -p1 -b .pcfix
 
 %build
 %ifarch %{ix86}
@@ -102,14 +101,6 @@ mv libNOTvpx_g.a libvpx_g.a
 %install
 make DIST_DIR=%{buildroot}%{_prefix} dist
 
-# Install the pkg-config file
-mkdir -p %{buildroot}%{_libdir}/pkgconfig/
-install -m0644 %{SOURCE1} %{buildroot}%{_libdir}/pkgconfig/
-# Fill in the variables
-sed -i "s|@PREFIX@|%{_prefix}|g" %{buildroot}%{_libdir}/pkgconfig/libvpx.pc
-sed -i "s|@LIBDIR@|%{_libdir}|g" %{buildroot}%{_libdir}/pkgconfig/libvpx.pc
-sed -i "s|@INCLUDEDIR@|%{_includedir}|g" %{buildroot}%{_libdir}/pkgconfig/libvpx.pc
-
 # Simpler to label the dir as %doc.
 mv %{buildroot}/usr/docs doc/
 
@@ -145,7 +136,6 @@ popd
 # These are SDK docs, not really useful to an end-user.
 %doc docs/html/
 %{_includedir}/vpx/
-%{_libdir}/pkgconfig/libvpx.pc
 %{_libdir}/pkgconfig/vpx.pc
 %{_libdir}/libvpx.so
 
@@ -153,6 +143,13 @@ popd
 %{_bindir}/*
 
 %changelog
+* Tue May 29 2012 Tom Callaway <spot at fedoraproject.org> - 1.0.0-3
+- fix vpx.pc file to include -lm (bz825754)
+
+* Fri May 11 2012 Tom Callaway <spot at fedoraproject.org> - 1.0.0-2
+- use included vpx.pc file (drop local libvpx.pc)
+- apply upstream fix to vpx.pc file (bz 814177)
+
 * Mon Jan 30 2012 Tom Callaway <spot at fedoraproject.org> - 1.0.0-1
 - update to 1.0.0
 


More information about the scm-commits mailing list