Bug 244356 - Review Request: xapian-bindings - Bindings for Xapian
Review Request: xapian-bindings - Bindings for Xapian
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Parag AN(पराग)
Fedora Package Reviews List
:
Depends On:
Blocks: 244357 244389
  Show dependency treegraph
 
Reported: 2007-06-15 05:13 EDT by Marco Pesenti Gritti
Modified: 2010-06-11 00:26 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-06-19 16:33:09 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
panemade: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Marco Pesenti Gritti 2007-06-15 05:13:56 EDT
Spec URL: http://dev.laptop.org/~marco/xapian-bindings.spec
SRPM URL: http://dev.laptop.org/~marco/xapian-bindings-0.9.10.1-2.2.svn8397.src.rpm
Description:
Xapian is an Open Source Probabilistic Information Retrieval Library. It offers a highly adaptable toolkit that allows developers to easily add advanced
indexing and search facilities to applications. This package provides the
files needed for scripts which use Xapian.
Comment 1 Olly Betts 2007-06-15 13:42:05 EDT
For the same reasons as for bug#244355, please package Xapian 1.0.1 instead.
Comment 3 Parag AN(पराग) 2007-06-19 05:41:56 EDT
E: xapian-bindings no-binary
The package should be of the noarch architecture because it doesn't contain
any binaries.
Comment 4 Parag AN(पराग) 2007-06-19 05:55:51 EDT
any reason to split docs in xapian-bindings ??
why not to include in in xapian-bindings-python
Comment 5 Marco Pesenti Gritti 2007-06-19 06:38:47 EDT
xapian-bindings also contains bindings for csharp, java, ruby etc. We are not
building them right now but we might want to do it in the future. I'd happily do
away with the doc only package, just not sure how to do it since the docs are
common to all the bindings.

About noarch. If I add "BuildArch: noarch" to the main package the build fails
because the debug info files are created but not packaged. I can only make it
work by adding "BuildArch: i386" to the python subpackage, which obviously is
wrong. Suggestions on how to fix it?
Comment 6 Marco Pesenti Gritti 2007-06-19 07:23:47 EDT
As discussed on irc, I moved the docs file to the python package. We are not
building the base xapian-bindings package anymore.

http://dev.laptop.org/~marco/xapian-bindings.spec
http://dev.laptop.org/~marco/xapian-bindings-1.0.1-1.src.rpm

Comment 7 Parag AN(पराग) 2007-06-19 08:01:15 EDT
remove 
Requires: %{name} = %{version}
and bump release and update links
Comment 9 Parag AN(पराग) 2007-06-19 08:27:20 EDT
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPMs.
+ source files match upstream.
3280974b85acf143d667866a8a4a4d7c  xapian-bindings-1.0.1.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
- dist tag is NOT present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is small so no need of -doc subpackage.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no .pc files are present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets used.
APPROVED.
Comment 10 Marco Pesenti Gritti 2007-06-19 08:32:48 EDT
New Package CVS Request
=======================
Package Name: xapian-bindings
Short Description: Bindings for the Xapian Information Retrieval Library
Owners: mpg@redhat.com
Branches: OLPC-2
InitialCC: mpg@redhat.com
Comment 11 Jason Tibbitts 2007-06-19 15:20:15 EDT
CVS done.
Comment 12 Marco Pesenti Gritti 2007-08-07 13:12:50 EDT
Package Change Request
======================
Package Name: xapian-bindings
New Branches: F-7
Updated Fedora Owners: mpg@redhat.com, adel.gadllah@gmail.com
Comment 13 Kevin Fenzi 2007-08-07 20:09:47 EDT
cvs done.
Comment 14 Peter Robinson 2010-06-10 15:52:03 EDT
Package Change Request
======================
Package Name: xapian-bindings
New Branches: EL-6
Owners: pbrobinson sdz
Comment 15 Kevin Fenzi 2010-06-11 00:26:20 EDT
cvs done.

Note You need to log in before you can comment on or make changes to this bug.