Bug 455210 - Review Request: odfpy - Python library for manipulating OpenDocument files
Review Request: odfpy - Python library for manipulating OpenDocument files
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ricky Zhou
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-13 23:13 EDT by Ian Weller
Modified: 2008-07-23 17:46 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-07-23 17:46:28 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rickyz: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Ian Weller 2008-07-13 23:13:40 EDT
Spec URL: http://ianweller.fedorapeople.org/SRPMS/odfpy/0.7-1/odfpy.spec
SRPM URL: http://ianweller.fedorapeople.org/SRPMS/odfpy/0.7-1/odfpy-0.7-1.fc9.src.rpm
Description: 

Odfpy aims to be a complete API for OpenDocument in Python. Unlike
other more convenient APIs, this one is essentially an abstraction
layer just above the XML format. The main focus has been to prevent
the programmer from creating invalid documents. It has checks that
raise an exception if the programmer adds an invalid element, adds an
attribute unknown to the grammar, forgets to add a required attribute
or adds text to an element that doesn't allow it.

These checks and the API itself were generated from the RelaxNG
schema, and then hand-edited. Therefore the API is complete and can
handle all ODF constructions, but could be improved in its
understanding of data types.
Comment 1 Ricky Zhou 2008-07-14 22:26:19 EDT
Build cleanly in mock.

* rpmlint clean
* sources match upstream:
a8e76163f9d328fe7b8b93617cdc945692758408  odfpy-0.7.tgz
* package meets naming guidelines
x doesn't use consistent macros: I'd go with %{optflags} and %{buildroot}
instead of $RPM_OPT_FLAGS and $RPM_BUILD_ROOT
* build root is ok
* correct/allowed license tag
x License text is added to the package -
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text says it
shouldn't be.  I'd take it out for now and bug upstream :-)
* Requires/BuildRequires are OK
* No %check section, but the tests don't seem completely functional at the
moment (bug upstream)
* Installs/uninstalls fine
* Permissions are fine

APPROVED (just fix those the above tiny issues)
Comment 2 Ian Weller 2008-07-14 22:33:37 EDT
Thanks.

New Package CVS Request
=======================
Package Name: odfpy
Short Description: Python library for manipulating OpenDocument files
Owners: ianweller, pfrields
Branches: F-8 F-9 EL-4 EL-5
InitialCC: [none]
Cvsextras Commits: yes
Comment 3 Ian Weller 2008-07-14 22:37:03 EDT
Oh, whoops. I set fedora-cvs to +. Silly me!
Comment 4 Kevin Fenzi 2008-07-15 12:52:24 EDT
cvs done.
Comment 5 Fedora Update System 2008-07-15 15:05:17 EDT
odfpy-0.7-2.fc8 has been submitted as an update for Fedora 8
Comment 6 Fedora Update System 2008-07-15 15:05:58 EDT
odfpy-0.7-2.fc9 has been submitted as an update for Fedora 9
Comment 7 Fedora Update System 2008-07-17 10:17:46 EDT
odfpy-0.7-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 8 Fedora Update System 2008-07-17 10:20:07 EDT
odfpy-0.7-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 9 Ricky Zhou 2008-07-23 17:46:28 EDT
Closing NEXTRELEASE, since it has been pushed.

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