Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 219119 - Review Request: pyscript - Postscript graphics with Python
Summary: Review Request: pyscript - Postscript graphics with Python
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michał Bentkowski
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
: 191218 (view as bug list)
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-12-11 05:49 UTC by Jef Spaleta
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-13 04:20:06 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jef Spaleta 2006-12-11 05:49:29 UTC
Spec URL: ftp://jspaleta%40thecodergeek.com@ftp.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript.spec

SRPM URL: 
ftp://jspaleta%40thecodergeek.com@ftp.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript-0.6-5.fc6.src.rpm

Description:
PyScript is a Python module for producing high quality postscript graphics.
Rather than use a GUI to draw a picture, the picture is programmed using
Python and the PyScript objects.


Note:
I am re-submitting this with the support of the project author.  I plan to have the project author as a co-maintainer, and will act as a gateway for cvs commits for him until such time as he gains fedora cvs access through sponsorship.

Comment 1 Jef Spaleta 2006-12-11 05:52:45 UTC
*** Bug 191218 has been marked as a duplicate of this bug. ***

Comment 2 Peter Gordon 2006-12-11 06:03:53 UTC
Eh, sorry Jef. The hosting is HTTP-only for anonymous/public access.

Spec: http://jspaleta.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript.spec
SRPM:
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript-0.6-5.fc6.src.rpm

Thanks for your understanding; and apologies for the slight inconvenience.

Comment 3 Jef Spaleta 2006-12-11 06:14:44 UTC
opps, I just cut and pasted the url through nautilus this time. I didn't think
about the ftp/http issue oh well. For everyone playing along, the urls in
Comment #2 are correct for public access.

-jef

Comment 4 Michał Bentkowski 2006-12-11 17:22:08 UTC
I'll do the review soon...

Comment 5 Michał Bentkowski 2006-12-11 18:15:43 UTC
MUST items:
!* rpmlint output:
W: pyscript macro-in-%changelog python_sitelib
W: pyscript macro-in-%changelog ghost
W: pyscript macro-in-%changelog __python
W: pyscript macro-in-%changelog _libdir
E: pyscript no-binary
E: pyscript-debuginfo empty-debuginfo-package
 * package is named well
 * spec file name is good
 * package meets Packaging Guidelines
 * package is licensed with a GPL open-source compatible license
 * License field in spec file matches actual license
 * license file is included in %doc
 * md5sums are matching (8b4945fda4318abdcd77f1573996b9da)
 * package successfully compiles on x86_64
 * BuildRequires listed well
 * no locales
 * no need to %post and %postun sections
 * not relocatable
 * package owns directories well
 * no duplicates in %files
 * %files section includes %defattr
 * proper %clean section
 * macros used well

THINGS to do:
 * get rid of all macro-in-%changelog rpmlint warnings by doubling all
% characters in %changelog. For example: instead of %{_libdir} write %%{_libdir}
 * fix no-binary and debuginfo package problems by adding BuildArch: noarch
to preamble
 * from my experience I know that using python-devel BR instead of python
is better solution 
 * do we really need tetex package? I have grepped all the sources and I found
that pyscript needs only latex application which is part of tetex-latex package.
Thus I believe that it is a dependency we really need. Correct me if I am wrong.


Comment 6 Jef Spaleta 2006-12-12 09:01:18 UTC
Okay I think the new srpm and spec cover everything in the TODO. rpmlint retunrs
clean for me now.

I read over the py files and it is calling latex and dvips which are both in the
tetex-latex package, not tetex.

http://jspaleta.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript.spec
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript-0.6-6.fc7.src.rpm
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/pyscript/pyscript-0.6-6.fc7.noarch.rpm

Comment 7 Michał Bentkowski 2006-12-12 17:28:38 UTC
rpmlint is silent now, so nothing stands in the way to make this package
accepted.

APPROVED.

Don't forget to close this ticket after rebuilding package in devel.


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