[Xapian-devel] updated .spec file for xapian (FC6)

[Olly Betts]

On Tue, Nov 07, 2006 at 11:19:26AM -0500, Neal Becker wrote:
> Summary: The Xapian Probabilistic Information Retrieval Library
> Name: xapian-core
> Version: 0.9.8
> Release: 2
> License: GPL
> Vendor: www.xapian.org
> Group: Applications/Databases
> URL: http://www.xapian.org/
> Requires: %{name}-libs = %{version}
> Source0: http://www.oligarchy.co.uk/xapian/%{version}/%{name}-%{version}.tar.gz
> BuildRequires: autoconf automake autoconf libtool

You have autoconf twice here!

> %package devel
> Group: Development/Libraries
> Summary: Files needed for building packages which use Xapian.
> BuildRequires: autoconf
> Requires: %{name}-libs = %{version}

I don't think we want a BuildRequires: here too do we?  This is just
specifying a binary package to be built from this source package (those
are the debian terms - sorry, but I don't recall the RPM nomenclature).

Otherwise it looks good to me, though I'm no RPM expert so it would be
good to have Fabrice Colin's input as he really maintains this spec file.
Fabrice: see the attached patch.

I think we also want to run "autoreconf --force" in xapian-omega's and
xapian-binding's spec files too for the same reason we do here.

Cheers,
    Olly
-------------- next part --------------
--- xapian.spec	2006-11-07 04:48:31.000000000 +0000
+++ xapian.spec.neal	2006-11-07 17:19:39.000000000 +0000
@@ -1,16 +1,17 @@
 # RedHat-style .spec file for Xapian
 # xapian.spec.  Generated from xapian.spec.in by configure.
 
-Summary: The Xapian Probabilistic Information Retrieval Library.
+Summary: The Xapian Probabilistic Information Retrieval Library
 Name: xapian-core
 Version: 0.9.8
-Release: 1
+Release: 2
 License: GPL
 Vendor: www.xapian.org
 Group: Applications/Databases
 URL: http://www.xapian.org/
 Requires: %{name}-libs = %{version}
 Source0: http://www.oligarchy.co.uk/xapian/%{version}/%{name}-%{version}.tar.gz
+BuildRequires: autoconf automake autoconf libtool
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root
 
 %description
@@ -31,6 +32,7 @@
 %package devel
 Group: Development/Libraries
 Summary: Files needed for building packages which use Xapian.
+BuildRequires: autoconf
 Requires: %{name}-libs = %{version}
 
 %description devel
@@ -43,6 +45,7 @@
 %setup -q -n %{name}-%{version}
 
 %build
+autoreconf --force
 %configure
 make
 
@@ -59,13 +62,18 @@
 mkdir -p %{buildroot}%{_datadir}/doc/%{name}-%{version}
 cp AUTHORS ChangeLog COPYING NEWS PLATFORMS README %{buildroot}%{_datadir}/doc/%{name}-%{version}
 
-%post -p /sbin/ldconfig
+%post libs -p /sbin/ldconfig
 
-%postun -p /sbin/ldconfig
+%postun libs -p /sbin/ldconfig
 
 %clean
 [ "%{buildroot}" != "/" ] && rm -rf %{buildroot}
 
+%changelog
+
+* Mon Nov  6 2006 Neal Becker <ndbecker2 at gmail.com> - 0.9.8-2
+- Add autoreconf to fix libtool for x86_64.
+
 %files
 %defattr(-, root, root)
 %{_bindir}/xapian-tcpsrv