Spec URL: http://ynemoy.fedorapeople.org/review/python-ekg.spec SRPM URL: http://ynemoy.fedorapeople.org/review/python-ekg-0.3.0-1.fc10.src.rpm Description: EKG is a community health scanner. Currently this targets mailing list archives, in the future, we may also pay closer attention to code contributions and other details. Note: We are in discussion what the name should be. There is a definite python library component. The big question is the scripts that come with this package, should they also be considered 'python-fedora', or would we be better off with a subpackage.
I am unable to get this package working in F10 with spython-sqlalchemy-0.4.8-1.fc10 Traceback (most recent call last): File "/usr/bin/ekg-scan", line 16, in <module> import ekg.model File "/usr/lib/python2.5/site-packages/ekg/model.py", line 112, in <module> Base = declarative_base(metadata=metadata, cls=SqlBase, metaclass=MetaSqlBase) TypeError: declarative_base() got an unexpected keyword argument 'cls'
I should do a better job packaging. This needs SA 0.5. Stay tuned for an updated package in the next couple of days.
Spec URL: http://ynemoy.fedorapeople.org/review/python-ekg.spec SRPM URL: http://ynemoy.fedorapeople.org/review/python-ekg-0.3.1-1.fc10.src.rpm Since i upgraded to setuptools along with fixing the dependencies, i decided this needed a new upstream release. Anyways, that should force the issue. If you need a 0.5.x SA package in F10, you should pull the SRPM from koji and rebuild it, because just installing from rawhide will try to pull the new python and everything else in too.
BTW, to anyone declaring intent to review, there is a new version upstream, ping here to let me know to upload the latest upstream.
- the description is "too long": make it a little paragraph if you will - mixed use of spaces and tabs in .spec - the Source0 should be a complete URL
Friendly ping to Yaakov ;-)
Spec URL: http://ynemoy.fedorapeople.org/review/python-ekg.spec SRPM URL: http://ynemoy.fedorapeople.org/review/python-ekg-0.4.0-2.fc10.src.rpm
Spec URL: http://ynemoy.fedorapeople.org/review/python-ekg.spec SRPM URL: http://ynemoy.fedorapeople.org/review/python-ekg-0.4.3-2.fc11.src.rpm
Minor error: $ rpmlint /home/jmeeuwen/rpmbuild/RPMS/noarch/python-ekg-0.4.3-2.fc11.noarch.rpm python-ekg.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/ekg/migration/manage.py 0644 1 packages and 0 specfiles checked; 1 errors, 0 warnings. Suggest to remove the shebang from that file if possible.
Setting the fedora-review flag as it seems to have been overlooked.
Seeing as this particular code -- python-ekg -- is not being actively worked on, I just wanted those reading this bug to realize that this package review, and our desire to get this code packaged into Fedora's repositories, is not urgent right now. If at some point we have more resources, or folks interested in python-ekg, perhaps we'll ressurect this package review. For the time being, the EKG project is trying to proceed in a different direction.
Max, are you suggesting that this review ticket should be closed out? I suppose you could leave it open if neither Jeroen nor Yaakov mind, but it sounds like there's not much point in proceeding at this point.
It's up to Yaakov and Max ;-)
The only reason I commented here yesterday anyway was for the sake of cleanliness, and in that same spirit, I think this review request can be closed, and re-opened or a new one created if necessary at a future point.
Removing myself for these bug components as I'm either no longer involved in that aspect of the project, or no longer care to watch this particular bug. Sorry if you are caught in a maelstrom of bug changes as a result!
Stalled review, SPEC and SRPM are not accessible anymore. Closing.