Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/defcon.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/defcon-0.1-1.fc13.src.rpm Description: A set of flexible objects for representing UFO data.
Following python naming guidelines now. Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/python-defcon.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/python-defcon-0.1-1.fc13.src.rpm
Review: Good: - name ok - source matches upstream: 738f62fea9d78b5225706dc7e1a562be - latest version packaged - $ rpmlint ./python-defcon-0.1-1.fc13.src.rpm noarch/python-defcon-0.1-1.fc13.noarch.rpm 2 packages and 0 specfiles checked; 0 errors, 0 warnings. Needswork: - *** Warning: defcon requires FontTools, see: fonttools.sf.net *** Warning: defcon requires RoboFab, see: robofab.com -> Missing BR: python-robofab and fonttools (last one also R) - Some tests are failing, does upstream know of this? Ran 78 tests in 0.641s FAILED (failures=5) - You could build the documentation and include that into a -doc subpackage. (I didn't found it on an upstream website.) - Are you sure, this is installed correctly? From __init__.py: from defcon.objects.font import Font 9 from defcon.objects.glyph import Glyph, addRepresentationFactory, removeRepr esentationFactory 10 from defcon.objects.contour import Contour But there is no defcon.objects. etc installed, just in the source... It seems python setuptools or the setup.py are wrong. :(
Forgot to add: - license ok - no *.la - buildarch noarch ok - no libs - no locales -> false installing and BR are the blockers, doc is should
When I asked upstream if they can help to fix this testcase failure issue, Author "Tal Leming" wrote me => No. The tests work for me, so it is probably a problem with your installation or your platform. I don't have time to debug that for you. In any case, the same license that allows you to redistribute this code also allows you to modify it. If these are indeed bugs, you can fix them for your repackaged version. If you submit the patches to me, I will consider applying them to my code.
updated package addressing remaining issues Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/python-defcon.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/python-defcon-0.1-2.fc13.src.rpm
(In reply to comment #2) > Needswork: > - *** Warning: defcon requires FontTools, see: > fonttools.sf.net > *** Warning: defcon requires RoboFab, see: > robofab.com > > -> Missing BR: python-robofab and fonttools (last one also R) Fixed > - Some tests are failing, does upstream know of this? > Ran 78 tests in 0.641s > FAILED (failures=5) Maybe they work, once it's installed correctly (see below). > - You could build the documentation and include that into a -doc subpackage. > (I didn't found it on an upstream website.) Fixed > - Are you sure, this is installed correctly? > From __init__.py: > from defcon.objects.font import Font > 9 from defcon.objects.glyph import Glyph, addRepresentationFactory, > removeRepr esentationFactory > 10 from defcon.objects.contour import Contour > > But there is no defcon.objects. etc installed, just in the source... > It seems python setuptools or the setup.py are wrong. :( $ python -c "import defcon" Traceback (most recent call last): File "<string>", line 1, in <module> File "/usr/lib/python2.6/site-packages/defcon/__init__.py", line 8, in <module> from defcon.objects.font import Font ImportError: No module named objects.font So this is not yet correctly installed. (Rest looks ok.)
I have no time to work on this. Closing this for now as deferred.