Bug 495925 - Review Request: pyxmlsec - Python bindings for the XML Security Library
Summary: Review Request: pyxmlsec - Python bindings for the XML Security Library
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Marek Mahut
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-04-15 15:25 UTC by Lubomir Rintel
Modified: 2009-04-16 05:39 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-04-16 05:39:04 UTC
Type: ---
Embargoed:
mmahut: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Rintel 2009-04-15 15:25:47 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/pyxmlsec.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/pyxmlsec-0.3.0-1.fc11.src.rpm

Description:

PyXMLSec is a set of Python bindings for the XML Security Library, a C
library based on LibXML2. The library supports major XML security standards
including, XML Signature, XML Encryption, Canonical XML and Exclusive
Canonical XML.

Comment 1 Lubomir Rintel 2009-04-15 15:27:11 UTC
Builds fine in el5 mock.
Rpmlint has one warning, which can be safely ignored:

pyxmlsec.i586: E: non-executable-script /usr/lib/python2.6/site-packages/xmlsec.py 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

It's not executable and not meant to be executable. Shebang there is therefore just a comment, and thus I'm not going to patch it away.

Comment 2 Marek Mahut 2009-04-15 16:05:49 UTC
Liar, liar, pants on fire!

+ iconv -f iso8859-1 -t utf8
/var/tmp/rpm-tmp.88790: line 42: /usr/share/doc/pyxmlsec-0.3.0/AUTHORS: No such file or directory
error: Bad exit status from /var/tmp/rpm-tmp.88790 (%prep)

http://koji.fedoraproject.org/koji/taskinfo?taskID=1300546

Comment 3 Lubomir Rintel 2009-04-15 16:08:42 UTC
(In reply to comment #2)
> Liar, liar, pants on fire!
> 
> + iconv -f iso8859-1 -t utf8
> /var/tmp/rpm-tmp.88790: line 42: /usr/share/doc/pyxmlsec-0.3.0/AUTHORS: No such
> file or directory
> error: Bad exit status from /var/tmp/rpm-tmp.88790 (%prep)

Doh! I forgot to make the package BuildRequire itself!

SPEC: http://v3.sk/~lkundrak/SPECS/pyxmlsec.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/pyxmlsec-0.3.0-2.fc11.src.rpm

Comment 4 Marek Mahut 2009-04-15 17:34:18 UTC
Looks fine, approved.

Comment 5 Lubomir Rintel 2009-04-15 17:54:03 UTC
Danke schon Marek!

New Package CVS Request
=======================
Package Name: pyxmlsec
Short Description: Python bindings for the XML Security Library
Owners: lkundrak
Branches: EL-5 F-10

Comment 6 Kevin Fenzi 2009-04-16 04:35:55 UTC
I assume you want a F-11 branch here. 

cvs done with F-11 branch added.

Comment 7 Lubomir Rintel 2009-04-16 05:39:04 UTC
Imported and built.


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