Bug 457207

Summary: Review Request: python-cssutils - CSS Cascading Style Sheets library for Python
Product: [Fedora] Fedora Reporter: Matthias Saou <matthias>
Component: Package ReviewAssignee: Michael Schwendt <bugs.michael>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: bugs.michael, fedora-package-review, notting
Target Milestone: ---Flags: bugs.michael: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-16 02:05:38 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 438609    

Description Matthias Saou 2008-07-30 09:22:54 UTC
Spec and SRPM URL: http://thias.fedorapeople.org/review/python-cssutils/
Description:
A Python package to parse and build CSS Cascading Style Sheets. DOM only, not
any rendering facilities.

Comment 1 Peter Robinson 2008-07-31 14:08:16 UTC
There looks to be a final 0.9.5 release available now.

Comment 2 Matthias Saou 2008-08-08 11:16:57 UTC
(In reply to comment #1)
> There looks to be a final 0.9.5 release available now.

I've just updated the package.

Comment 3 Matthias Saou 2008-08-19 13:32:02 UTC
* Tue Aug 19 2008 Matthias Saou <http://freshrpms.net/> 0.9.5.1-1
- Update to 0.9.5.1.

Comment 4 Michael Schwendt 2008-10-07 16:43:54 UTC
* Licence has changed:
cssutils is published under the LGPL 3 or later (from 0.9.5b3, before it was the LGPL 2.1) 

* rpmlint reports wrong-file-end-of-line-encoding for %doc files

* The /usr/bin/css* executables "Requires: python-setuptools" or else
they traceback!

* Group "Applications/Multimedia"? Rather Development/Libraries, but
the groups are too limited.

* The generated html documentation for the API could be put
into a separate -docs package.

Rest looks fine.

Comment 5 Matthias Saou 2008-10-09 15:40:11 UTC
(In reply to comment #4)

> * Licence has changed:

Good catch!

> * rpmlint reports wrong-file-end-of-line-encoding for %doc files

I had left out the removing of *.tmp files in the hack I had made to convert docs to the right EOF, those were probably the ones reported as having it wrong and it should be fixed now.

> * The /usr/bin/css* executables "Requires: python-setuptools" or else
> they traceback!

Another good catch!

> * Group "Applications/Multimedia"? Rather Development/Libraries, but
> the groups are too limited.

True. Changed.

> * The generated html documentation for the API could be put
> into a separate -docs package.

It's only 100kB uncompressed once the *.tmp files are no longer there, so not worth it IMHO.

Thanks a lot for the review. I've put python-cssutils-0.9.5.1-2 in the same location.

Comment 6 Michael Schwendt 2008-10-10 11:18:31 UTC
* BuildRequires: python-setuptools

is needed, too, or else it complains and starts downloading it
from python.org:

* I refer to the epydoc generated files in the "doc" directory:

$ du -h doc
14M     doc

And >1M compressed. *If* to be packaged, they must go into a -doc
sub-package according to the review guidelines.

* That's stuff that can be added in cvs, however, so:

APPROVED: python-cssutils-0.9.5.1-2

Comment 7 Matthias Saou 2008-10-10 12:03:33 UTC
All this is now fixed in 0.9.5.1-3 which is in the same location in case you want to have a quick look just in case.

Comment 8 Matthias Saou 2008-10-10 12:06:00 UTC
New Package CVS Request
=======================
Package Name: python-cssutils
Short Description: CSS Cascading Style Sheets library for Python
Owners: thias
Branches: F-9
InitialCC:

Comment 9 Kevin Fenzi 2008-10-10 23:06:39 UTC
cvs done.

Comment 10 Fedora Update System 2008-10-13 09:15:25 UTC
python-cssutils-0.9.5.1-3.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/python-cssutils-0.9.5.1-3.fc9

Comment 11 Fedora Update System 2008-10-16 02:05:35 UTC
python-cssutils-0.9.5.1-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.