[Bug 244356] New: Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

Summary: Review Request: xapian-bindings - Bindings for Xapian
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]

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.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review?

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |244389
              nThis|                            |

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2007-06-15 13:42 EST -------
For the same reasons as for bug#244355, please package Xapian 1.0.1 instead.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-18 15:56 EST -------
Updated to 1.0.1

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

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Product|Fedora Extras               |Fedora

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |244357
              nThis|                            |

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 05:41 EST -------
E: xapian-bindings no-binary
The package should be of the noarch architecture because it doesn't contain
any binaries.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 05:55 EST -------
any reason to split docs in xapian-bindings ??
why not to include in in xapian-bindings-python

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 06:38 EST -------
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?

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 07:23 EST -------
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

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 08:01 EST -------
remove 
Requires: %{name} = %{version}
and bump release and update links

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

------- Additional Comments From [e-mail]  2007-06-19 08:16 EST -------
Done.

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

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2007-06-19 08:27 EST -------
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.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2007-06-19 08:32 EST -------
New Package CVS Request
=======================
Package Name: xapian-bindings
Short Description: Bindings for the Xapian Information Retrieval Library
Owners: [e-mail]
Branches: OLPC-2
InitialCC: [e-mail]

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2007-06-19 15:20 EST -------
CVS done.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
               Flag|fedora-cvs+                 |fedora-cvs?

------- Additional Comments From [e-mail]  2007-08-07 13:12 EST -------
Package Change Request
======================
Package Name: xapian-bindings
New Branches: F-7
Updated Fedora Owners: [e-mail], [e-mail]

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xapian-bindings - Bindings for Xapian

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244356

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2007-08-07 20:09 EST -------
cvs done.

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=244356

Peter Robinson  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
               Flag|fedora-cvs+                 |fedora-cvs?

--- Comment #14 from Peter Robinson  2010-06-10 15:52:03 EDT ---
Package Change Request
======================
Package Name: xapian-bindings
New Branches: EL-6
Owners: pbrobinson sdz

[Bug 244356] Review Request: xapian-bindings - Bindings for Xapian

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=244356

Kevin Fenzi  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

--- Comment #15 from Kevin Fenzi  2010-06-11 00:26:20 EDT ---
cvs done.