[Xapian-devel] patch for xapian-spec(0.9.9)
Neal Becker
ndbecker2 at gmail.com
Thu Nov 9 17:31:42 GMT 2006
There were only 2 important parts:
add man page
remove '.' from summary (rpmlint wants that)
The others you can ignore. Sloppy diffing on my part.
On Thursday 09 November 2006 12:28 pm, Olly Betts wrote:
> On Thu, Nov 09, 2006 at 09:17:14AM -0500, Neal Becker wrote:
> > Missed %{_mandir}/man1/xapian-progsrv.1*
>
> Sigh, I even thought about this but incorrectly "remembered" that we
> had a glob for all the man pages in the .spec file (must have been
> thinking of the glob to get them whether they are gzipped or not).
>
> > -Summary: The Xapian Probabilistic Information Retrieval Library.
> > +Summary: The Xapian Probabilistic Information Retrieval Library
>
> Oops, I missed that change.
>
> > Name: xapian-core
> > Version: 0.9.9
> > -Release: 1
> > +Release: 3
>
> Why 3? The previous patched .spec file was for 0.9.8, this is 0.9.9.
> Not that it matters really.
>
> > %build
> > -# FC6 (at least) has a patched libtool which knows not to set rpath for
> > -# /usr/lib64, which upstream libtool fails to do currently. We can drop
> > -# this "autoreconf --force" and the "BuildRequires:" for the autotools
> > -# once upstream libtool is fixed. Note: this overwrites INSTALL, but
> > -# that doesn't matter here as we don't package it.
> > autoreconf --force
> > %configure
>
> Does this comment cause problems? I'd rather clearly document the
> purpose (and any side-effects) of such workarounds so they get removed
> when they are no longer required, rather than being left in place
> because nobody is sure exactly why they are done in the first place.
>
> Cheers,
> Olly
More information about the Xapian-devel
mailing list