Bug 427996 - Review Request: xqilla - XQilla is an XQuery and XPath 2.0 library, built on top of Xerces-C
Summary: Review Request: xqilla - XQilla is an XQuery and XPath 2.0 library, built on ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-08 15:58 UTC by Milan Zázrivec
Modified: 2009-01-04 20:48 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-11 14:55:23 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Milan Zázrivec 2008-01-08 15:58:35 UTC
Spec URL: http://www.fi.muni.cz/~xzazriv/xqilla/xqilla.spec
SRPM URL: http://www.fi.muni.cz/~xzazriv/xqilla/xqilla-2.0.0-0.fc8.src.rpm
Description:
XQilla is an XQuery and XPath 2.0 implementation written in C++ and based
on Xerces-C. It implements the DOM 3 XPath API, as well as having it's own
more powerful API. It conforms to the W3C proposed recomendation of XQuery
and XPath 2.0.

Comment 2 Mamoru TASAKA 2008-01-09 17:28:09 UTC
Before checking this package, would you confirm the license of
the following file(s) to upstream developer?
I guess the upstream forgot to change the license.

-------------------------------------------------------------------
src/mapm/mapm_mt.cpp
-------------------------------------------------------------------

Comment 3 Mamoru TASAKA 2008-01-09 17:31:04 UTC
(In reply to comment #2)
> would you confirm the license of
> the following file(s) to upstream developer?

Umm.. I used a strange English... Anyway please ask the
license of the files above to upstream.


Comment 4 Mamoru TASAKA 2008-01-09 17:39:47 UTC
For 2.0.0-0:

* Source
  - XQilla 2.0.0 source coincides, however for xerces_c source file,
    the tarball in your srpm is different from what I downloaded from
    the URL written as Source0:
----------------------------------------------------------
7760004 2005-09-08 23:14 xerces-c-src_2_7_0.tar.gz
8031146 2007-11-11 06:03 xqilla-2.0.0-0.fc9/xerces-c-src_2_7_0.tar.gz
----------------------------------------------------------

* Parallel make
  - If this is parallel make unsafe, please write a comment on
    the spec file.

Other things are okay, except for license issue.

Comment 5 Milan Zázrivec 2008-01-10 12:56:59 UTC
The problem with src/mapm/mapm_mt.cpp is a little more complicated. It is
a wrapper around MAPM library originally written by Martin Pfingstl. XQilla
uses MAPM library version 4.6.1, which did not include mapm_mt.cpp (it was 
distributed separately). At some point mapm_mt.cpp got included into MAPM
library and now the new MAPM license applies to mapm_mt.cpp too.

I have a written permission of the MAPM library author to distribute mapm
version 4.6.1 under a new license, but I don't think it applies to mapm_mt.cpp
(the one that is part of xqilla sources).

I removed src/mapm/mapm_mt.cpp from the original xqilla source tarball and
included mapm_mt.c (modified) from last upstream version of mapm into
the source rpm.

I can try to iron this problem out with the xqilla upstream, but hopefully
my solution could work in the meantime.

Source rpm + spec file:
* http://www.fi.muni.cz/~xzazriv/xqilla/xqilla-2.0.0-1.fc8.src.rpm
* http://www.fi.muni.cz/~xzazriv/xqilla/xqilla.spec

Scratch builds:
* http://koji.fedoraproject.org/koji/taskinfo?taskID=339227
* http://koji.fedoraproject.org/koji/taskinfo?taskID=339228
* http://koji.fedoraproject.org/koji/taskinfo?taskID=339229

Comment 6 Mamoru TASAKA 2008-01-10 15:14:29 UTC
Please replace 
----------------------------------------------------
make -j
----------------------------------------------------
with
----------------------------------------------------
make %{?_smp_mflags}
----------------------------------------------------
(per packaging guidelines). Other things are okay.


----------------------------------------------------
    This package (xqilla) is APPROVED by me
----------------------------------------------------

Comment 7 Milan Zázrivec 2008-01-10 15:23:42 UTC
New Package CVS Request
=======================
Package Name: xqilla
Short Description: XQilla is an XQuery and XPath 2.0 library, built on top of
Xerces-C
Owners: mzazrive
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: yes

Comment 8 Kevin Fenzi 2008-01-10 16:47:28 UTC
cvs done.

Comment 9 Mamoru TASAKA 2008-01-11 14:55:23 UTC
Closing.

Comment 10 Milan Zázrivec 2008-12-31 18:19:49 UTC
Package Change Request
======================
Package Name: xqilla
New Branches: EL-5
Owners: mzazrive

Comment 11 Kevin Fenzi 2009-01-04 20:48:32 UTC
cvs done.


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