[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12759423#action_12759423
 ] 

Henry Robinson commented on ZOOKEEPER-420:
------------------------------------------

Hi Mahadev - 

Thanks for helping out with this. I've added the compile dependency, and also 
fixed the "dirname: too few arguments" problem that you get if you try to 
manually run the tests without having run ant compile. 

For the last problem - could you add this line to run_tests.sh after the 
PYTHONPATH is set?

python -c "import sys; print sys.path"

I get:

['', '/Library/Python/2.5/site-packages/virtualenv-1.3.3-py2.5.egg', 
'/Library/Python/2.5/site-packages/simplejson-2.0.9-py2.5-macosx-10.5-i386.egg',
 
'/Users/henry/src/cloudera/apache-zookeeper/build/contrib/zkpython/lib.macosx-10.5-i386-2.5',
 '/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python25.zip', 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5', 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/plat-darwin',
 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/plat-mac',
 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/plat-mac/lib-scriptpackages',
 '/System/Library/Frameworks/Python.framework/Versions/2.5/Extras/lib/python', 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/lib-tk',
 
'/System/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/lib-dynload',
 '/Library/Python/2.5/site-packages', 
'/System/Library/Frameworks/Python.framework/Versions/2.5/Extras/lib/python/PyObjC']

The third directory is the one that the script should have found for the .so 
file on my machine. Do you see the PYTHONPATH directory in the list you get?




> build/test should not require install in zkpython
> -------------------------------------------------
>
>                 Key: ZOOKEEPER-420
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-420
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: contrib-bindings
>            Reporter: Patrick Hunt
>            Assignee: Henry Robinson
>             Fix For: 3.3.0
>
>         Attachments: build.jpg, ZOOKEEPER-420.patch, ZOOKEEPER-420.patch, 
> ZOOKEEPER-420.patch
>
>
> Currently you cannot just build and test the zkpython contrib, you need to 
> actually install the zookeeper client c library as well
> as the zkpython lib itself.
> There really needs to be 2 steps:
> 1) build/test zkpython "encapsulated" within the src repository, there should 
> be no requirement to actually install anything
> (this is esp the case for automated processes and for review by PMC during 
> release time for example)
> 2) build an egg that can be distributed/installed by end user

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to