Spec URL: http://people.redhat.com/lmacken/python-pastedeploy.spec SRPM URL: http://people.redhat.com/lmacken/python-pastedeploy-0.5-1.src.rpm Description: This tool provides code to load WSGI applications and servers from URIs; these URIs can refer to Python Eggs for INI-style configuration files. PasteScript provides commands to serve applications based on this configuration file.
The %doc section should better be %doc docs/* There is an inconsistent use of rpm macros. I don't know if the name should be python-paste-script or python-pastescript. The guidelines don't cover the case of python submodules. If it is "This makes a package name format of python-$NAME" it seems to be python-pastescript, but if one follow "use the name of the module that you type to import it in a script" it should be python-paste-script. The second naming convention is also consistent with perl modules naming. Maybe this deserves a message on fedora-extras-list?
The %doc section should better be %doc docs/* There is an inconsistent use of rpm macros. I don't know if the name should be python-paste-deploy or python-pastedeploy. The guidelines don't cover the case of python submodules. If it is "This makes a package name format of python-$NAME" it seems to be python-pastedeploy, but if one follow "use the name of the module that you type to import it in a script" it should be python-paste-deploy. The second naming convention is also consistent with perl modules naming. Maybe this deserves a message on fedora-extras-list?
Good call. I fired a quick mail off to fedora-extras-list about this. We'll see what comes up. https://www.redhat.com/archives/fedora-extras-list/2006-July/msg00676.html
http://people.redhat.com/lmacken/python-paste-deploy-0.5-2.src.rpm http://people.redhat.com/lmacken/python-paste-deploy.spec * Mon Jul 24 2006 Luke Macken <lmacken> - 0.5-2 - Rename package to python-paste-deploy - Fix docs inclusion - Fix inconsistent use of buildroots
I don't see any issue, so I'll wait for the final word on python subpackage naming issue before approving. I assign the bug to me since the review is almost done.
So, the name is up to you. In the mean time the guidelines about ghosting .pyo have somehow changed, it seems like they shouldn't be ghosted now. As it is still a draft, I don't make that a blocker. APPROVED
* Mon Aug 21 2006 Luke Macken <lmacken> - 0.5-3 - Include .pyo files instead of ghosting them. Imported into CVS, added to owners.list, FC-5 CVS branch requested, tagged and built for devel. Thanks!
Changed summary for tracking purposes.