Red Hat Bugzilla – Bug 243571
Review Request: python-mechanoid - Python Programmatic Web Browser
Last modified: 2007-11-30 17:12:07 EST
Spec URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-mechanoid.spec
SRPM URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-mechanoid-0.6.9-1.fc7.src.rpm
Description: mechanoid is a programmatic browser written in Python. It is not a
web-bot and does not parse /robots.txt. It is intended to function as
the backend of special browsers, such as voice-text/text-voice browsers,
or to perform directed tasks, such as login to SourceForge to do a Quick
Release or to send and receive Yahoo! web mail.
This failed to build for me:
+ /usr/bin/python setup.py install -O1 --skip-build --root
error: invalid Python installation: unable to open
/usr/lib64/python2.5/config/Makefile (No such file or directory)
error: Bad exit status from /var/tmp/rpm-tmp.12975 (%install)
You need a build dependency on python-devel. Adding it gets things building.
W: python-mechanoid wrong-file-end-of-line-encoding
and indeed that file seems to have come from DOS and needs fixing.
The package has a test suite, and it almost runs, but because DNS is not
configured in mock, this fails:
which makes a single test (out of 116) fail. It may be worth patching out to
get some test coverage, but then it might become a pain because nothing stops
them from adding more network-using tests in the future.
If you don't want to add the test suite, do go ahead and add a %check section
with the line for running the tests commented out so that folks who want to run
it themselves can easily do so.
* source files match upstream:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
X BuildRequires missing python-devel.
* %clean is present.
* package builds in mock (development, x86_64) after adding missing BR.
* package installs properly
X rpmlint has a valid complaint.
* final provides and requires are sane:
python-mechanoid = 0.6.9-1.fc8
python(abi) = 2.5
* %check is not present; one test cannot be run in mock.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
Thanks for the feedback.
I've made the changes you recommended. I changed the test with a sed
replacement to make it work without requiring net access. I will report these
to upstream so hopefully they are corrected in the future.
I see you patched the test but you still need something to call it:
Also, you need to go the other way with the COPYING file; not unix2dos but
dos2unix. Actually, it's easier to do it with sed:
sed -i 's/\r//' COPYING
That saves you the dos2unix dependency as well.
Good catch on the unix2dos. I've added the above two items.
Heh, actually you didn't bump the release for that one, which annoys rpmlint a
bit. Also, don't forget to double '%' in your changelog.
Anyway, you've fixed any substantive complaints I have; just be sure your
version-release matches your changelog entry and fix up that percent sign when
you check in.
New Package CVS Request
Package Name: python-mechanoid
Short Description: Python Programmatic Web Browser
Branches: FC-6 F-7
Package Change Request
Package Name: python-mechanoid
New Branches: EL-5
I forgot to request branches for EPEL in previous request.
Please don't forget to close this ticket once the package has been imported and
It appears that the source has never been imported into the cvs repository or built.
Thanks for the ping. I've gotten it built and it's ready to go.