Spec Name or Url: http://linux.duke.edu/~icon/misc/fe/python-astng.spec SRPM Name or Url: http://linux.duke.edu/~icon/misc/fe/python-astng-0.13.0-0.1.src.rpm Description: The aim of this module is to provide a common base representation of python source code for projects such as pychecker, pyreverse, pylint, and others. It extends the class defined in the compiler.ast python module with some additional methods and attributes. PS: This is pulled in as a newly introduced requirement for pylint-0.8.
Updated to version 0.13.1 released today. http://linux.duke.edu/~icon/misc/fe/python-astng.spec http://linux.duke.edu/~icon/misc/fe/python-astng-0.13.1-0.1.src.rpm
Is this line really required? Requires: python-abi = %(%{__python} -c "import sys ; print sys.version[:3]") I thought it was unnecessary for FC4+ Other than that it looks OK, I am building it using mock, and if all goes as I expect I will approve it soon. :-)
OK, ignore my comment, I just saw that line in fedora-rpmdevtools rpm.
Review: + it build in mock on x86_64 + it carries the license, the license is correct (GPL) and valid for Extras + the source file is the same as upstream + the rpm follows the packaging guides + the rpm follows the packaging name rules + rpmlint does not have any output in any of the generated rpms + the spec file is readable and it is written in American English + the package carries code not contents Approved.
Thank you, kind sir -- it will be imported shortly. :)