Temporarily update README install instructions #47

Merged
merged 1 commit into from Dec 14, 2016

Conversation

Projects
None yet
4 participants
Owner

johnsca commented Dec 14, 2016

Until libjuju is released to pypi, our instructions need to include
installing using the wheelhouse. Hopefully this can be reverted very
soon.

README.md
- sudo pip3 install 'git+https://github.com/juju-solutions/matrix.git'
+ git clone https://github.com/juju-solutions/matrix.git
+ cd matrix
+ sudo pip3 install . -f wheelhouse
@petevg

petevg Dec 14, 2016

Collaborator

I don't like telling people to install in their system Python. :-/

This command (minus the sudo) fails in a virtualenv, as it can't find zookeeper.h (probably a lib dir missing from the include path).

@petevg

petevg Dec 14, 2016

Collaborator

Would prefer:

git clone https://github.com/juju-solutions/matrix.git
cd matrix
tox -r 
source .tox/py35/bin/activate
@johnsca

johnsca Dec 14, 2016

Owner

There are instructions for installing in a venv further down. The point of the default install instructions is to install at the system (or container) level.

A snap for this would be good, but it would have to be unconfined. It's worth looking in to, though, so I created #48

@bcsaller

bcsaller Dec 14, 2016

Contributor

zookeeper.h sounds like its pulling the old python (pre-juju 1.0) bindings that still are in pip and called juju. If you have that sitting around somewhere it would try to build the zookeeper bindings, but nothing modern uses it

This still fails (on a freshly minted Xenial LXD with python3-pip and git installed)

http://paste.ubuntu.com/23630502/

Temporarily update README install instructions
Until libjuju is released to pypi, our instructions need to include
installing using the wheelhouse.  Hopefully this can be reverted very
soon.
Owner

johnsca commented Dec 14, 2016

I was missing the --no-index to prevent it from picking up the old juju lib. Please try again.

Thanks

- sudo pip3 install 'git+https://github.com/juju-solutions/matrix.git'
+ git clone https://github.com/juju-solutions/matrix.git
+ cd matrix
+ sudo pip3 install . -f wheelhouse --no-index
@petevg

petevg Dec 14, 2016

Collaborator

Hooray for working things!

Would still prefer evangelizing virtualenvs in all the places, but I can see cutting it out for simplicity :-)

Collaborator

petevg commented Dec 14, 2016

+1, as --no-index solved all my problems.

Works much better. Confirmed I can install matrix on a fresh Xenial LXD following these instructions.

@petevg petevg merged commit 7cb5efa into master Dec 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment