[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