Bug 237354 - Review Request: python-meld3 - An HTML/XML templating system for Python
Summary: Review Request: python-meld3 - An HTML/XML templating system for Python
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Toshio Kuratomi
QA Contact: Fedora Package Reviews List
Depends On:
Blocks: 237344
TreeView+ depends on / blocked
Reported: 2007-04-20 23:52 UTC by Mike McGrath
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-04-22 16:20:30 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
toshio: fedora-review+

Attachments (Terms of Use)
Patch to build on x86_64 (1005 bytes, patch)
2007-04-22 00:26 UTC, Toshio Kuratomi
no flags Details | Diff

Description Mike McGrath 2007-04-20 23:52:48 UTC
Spec: http://mmcgrath.net/~mmcgrath/meld3/python-meld3.spec
SRPM: http://mmcgrath.net/~mmcgrath/meld3/python-meld3-0.6-1fc6.src.rpm
meld3 is an HTML/XML templating system for Python 2.3+ which keeps template
markup and dynamic rendering logic separate from one another. See
http://www.entrian.com/PyMeld for a treatise on the benefits of this pattern.

Comment 1 Toshio Kuratomi 2007-04-22 00:26:15 UTC
Created attachment 153247 [details]
Patch to build on x86_64

778b2ac28cbdc3932774246791884973  python-meld3-0.6-1fc6.src.rpm

* Does not build on x86_64.  Since the module includes a binary module, it ends

  up in sitearch (%{_libdir}/python2.?/site-packages, rather than /usr/lib).
  We can use the sitearch macro instead of the sitelib macro to find this.
  Patch for spec file attached.

Should do:
* meld3 doesn't contain a file with the license in it.	Should email the author

  to add that in his next tarball.

* rpmlint output: 
  W: python-meld3 invalid-license ZPL
  W: python-meld3 invalid-license ZPL
  W: python-meld3-debuginfo invalid-license ZPL
  ZPL is the Zope Public License, This is ignorable.

* Package follows package naming guidelines.
* Spec file named appropriately.
* Package licensed under the Zope Public License v2 which is FSF approved.
* Spec file is legible American English.
* Source matches upstream.
* All build requirements are listed.
* No locale files.
* Not a dynamic library.
* Not relocatable.
* Package owns all files nad directories it installs.
* Package does not own files owned by other packages.
* %files has a proper defattr line.
* Consistent use of macros.
* Code not content.

With the attached patch, this package is APPROVED.

Comment 2 Mike McGrath 2007-04-22 16:20:30 UTC
Imported and building.  Updates were made and upstream has been notified about
license issue.

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