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 

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

I get:

['', '/Library/Python/2.5/site-packages/virtualenv-1.3.3-py2.5.egg', 

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