[Bug 463217] Review Request: gnuplot-py - Python interface to Gnuplot

bugzilla at redhat.com bugzilla at redhat.com
Fri Nov 28 22:57:48 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=463217





--- Comment #11 from Orcan 'oget' Ogetbil <orcanbahri at yahoo.com>  2008-11-28 17:57:47 EDT ---
Sorry, I was never notified that you updated the package. Anyways, I found a
few issues:

* There are broken links in the %doc html files. For example, in your browser,
go to /usr/share/doc/gnuplot-py-1.8/gp_unix.html . The Gnuplot link on the top
left is broken, which is the case for pretty much all other %doc files. Can you
fix those?

* I also ask you the include the other relevant doc files (*.txt) in the %doc .
These .txt files are referred to within the .html files (see Gnuplot.html) and
therefore we need to keep them.

You can put those happydoc generated .html files in a subdirectory inside
/usr/share/doc/gnuplot-py-1.8/ if you think that having everything in the main
%doc directory makes things confusing.

* This python package does not use setuptools. Therefore in order to create the
proper python egg, you need to follow the guideline at:
http://fedoraproject.org/wiki/Packaging/Python/Eggs#Providing_Eggs_for_non-setuptools_packages

* You don't need to package:
   /usr/lib/python2.5/site-packages/Gnuplot/setup.py*
   /usr/share/doc/gnuplot-py-1.8/setup.html
files. There are also mac, win32 etc related .py and .html files. Those aren't
very necessary either. I leave these up to you.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list