Bug 247268
Summary: | Review Request: python-proctor - Proctor is a tool for running unit tests | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Anderson Silva <ansilva> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED CANTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-08-13 15:43:00 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Anderson Silva
2007-07-06 14:33:47 UTC
Just to show the output of rpmlint on this package: [ansilva@dhcp227-94 noarch]$ rpmlint python-proctor-1.2-2.noarch.rpm [ansilva@dhcp227-94 noarch]$ This person is in need of a sponsor. * Package passes rpmlint [steve@psycho Desktop]$ rpmlint /home/steve/rpmbuild/RPMS/noarch/python-proctor-1.2-2.fc7.noarch.rpm [steve@psycho Desktop]$ * Package passes Package Naming Guidelines * %{name} is the same as spec file * Meets the Packaging Guidelines * Licensed under BSD. NOTE: license is not in upsteam source. Please notify the owner and request it be added. * The License field in the package spec file matches the actual license. * spec file is written in American English. * spec is legible * Upstream matches whats in srpm [steve@psycho Desktop]$ md5sum Proctor-1.2.tar.gz* cee63679980ea25816511bb1b8be0d7f Proctor-1.2.tar.gz cee63679980ea25816511bb1b8be0d7f Proctor-1.2.tar.gz-orig [steve@psycho Desktop]$ * Package compiles in noarch * BuildRequires looks sane. * Build doesn't have duplicate files listed * %files look sane * %clean is sane * macros used throughout spec * package is python libraries which is code * Package does not require docs to run X Unclear if gui.py should be available or if it is in progress. If it should work then a %{name}.desktop file is needed and that file must be properly installed with desktop-file-install in the %install section. Please talk with the developer. * Does not own files or directories owned by other packages * %install has rm -rf $RPM_BUILD_ROOT * Files are all valid UTF-8 X Package seems to require Pmw (http://pmw.sourceforge.net/ I believe). Without Pmw warnings are thrown on running tests and gui does not work. [steve@psycho rpmbuild]$ python /usr/lib/python2.5/site-packages/proctorlib/gui.py Traceback (most recent call last): File "/usr/lib/python2.5/site-packages/proctorlib/gui.py", line 38, in <module> import Pmw ImportError: No module named Pmw [steve@psycho rpmbuild]$ - Note: A later version has been released -- 1.3 Mon Jul 30 07:43:36 2007 (http://www.doughellmann.com/projects/Proctor/) * Package seems to work as expected with the noted exception of the missing GUI lib: [steve@psycho proctorlib]$ proctorbatch --list WARNING: Could not import icon (No module named PmwContribD) test: tests.ExampleTestCase.test00 test: tests.ExampleTestCase.test01 test: tests.ExampleTestCase.test02 test: tests.ExampleTestCase.test03Failure test: tests.ExampleTestCase.test04Error test: tests.ExampleTestCase.test05 test: tests.ExampleTestCase.test06 test: tests.ExampleTestCase.test07 test: tests.ExampleTestCase.test08Failure test: tests.ExampleTestCase.test09Error test: tests.ExampleTestCase.test10 test: tests.ExampleTestCase.test11 test: tests.ExampleTestCase.test12 .... It also is importable as expected .... [steve@psycho proctorlib]$ python Python 2.5 (r25:51908, Apr 10 2007, 10:29:13) [GCC 4.1.2 20070403 (Red Hat 4.1.2-8)] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import proctorlib >>> print proctorlib.__doc__ Library for Proctor. I've resolved to move along to a new package, I will try to get someone else to pick this package up. |