Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 176683 - dev-python/cheetah-2.0_rc8 fails on tests
Summary: dev-python/cheetah-2.0_rc8 fails on tests
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Python Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-01 13:29 UTC by Thomas Anderson (tanderson) (RETIRED)
Modified: 2007-05-01 19:17 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
cheetah-2.0_rc8.ebuild (cheetah-2.0_rc8.ebuild,1.10 KB, text/plain)
2007-05-01 17:34 UTC, Ali Polatel (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2007-05-01 13:29:28 UTC
Cheetah fails on tests because it tries to run the "cheetah" executable which is not in the $PATH. This is because, at the time of src_test cheetah has not yet been installed.

Reproducible: Always

Steps to Reproduce:
1.FEATURES="test" emerge dev-python/cheetah
2.
3.

Actual Results:  
Fails on tests because it can't find cheetah executable
Comment 1 Ali Polatel (RETIRED) gentoo-dev 2007-05-01 17:34:05 UTC
Created attachment 117861 [details]
cheetah-2.0_rc8.ebuild

The problem is about the self.go("cheetah .. calls in
src/Tests/CheetahWrapper.py. It assumes cheetah is in PATH.
Setting PATH to ${PATH}:$(ls -d ./build/scripts*) before calling
python doesn't work.

So a logical fix is editing self.go("cheetah .. calls by setting
cheetah to $(ls -d ./build/scripts*)/cheetah and adding 
PYTHONPATH=$(ls -d ./build/lib.*) before them which worked fine here
and cheetah passed all the tests.

Attached ebuild does that..
Comment 2 Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2007-05-01 18:39:50 UTC
Proposed ebuild fixes bug on ~amd64
Comment 3 Rob Cakebread (RETIRED) gentoo-dev 2007-05-01 18:52:23 UTC
Thanks for the sed-work Ali, works great and thanks for reporting, Thomas.
Fixed in CVS.
Comment 4 Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2007-05-01 19:17:09 UTC
Ugg, I just missed it. I was posting an ebuild and we collided. Yours was better anyway ;-)