[Bug 806040] Review Request: pyproj - a python module that performs cartographic transformations and geodetic computations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--- Comment #17 from jdekloe <kloe0040@xxxxxxxxx> ---
Thanks for your comments.
I have added the download date to the release version as requested.

As for the TODO comments:

#TODO: Consider Sphinx for documentation

this was an idea added by Volker Fröhlich in his original rpm spec file. I
don't know Sphinx from own experience, but from what I read the output seems to
be nicer formatted. However, documentation generated by epydoc is already
present in the source tree, so for me this means the module is in a usable
state, since it includes api documentation. Therefore I removed this TODO
comment and would suggest to pass the request to use sphinx to the upstream
author for consideration.

#TODO: pyproj.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/pyproj/_proj.so 0775L

this problem has been fixed now, so I removed this TODO as well.

Finally, the #TODO in the check section is more a reminder that the source code
does not include a proper test script. It is not a TODO to be included in the
spec file, so I turned it into a normal comment. Also adding a proper test
script should be suggested to the upstream author. If you agree I am willing to
do that. In the meantime I hope this is no reason to keep this module from
inclusion in Fedora.

New spec and srpm files are here:

Spec URL: http://jdekloe.nl/Fedora/pyproj.spec
SRPM URL: http://jdekloe.nl/Fedora/pyproj-1.9.2-5.20120712svn300.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]