Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Brew install #431

Merged
merged 5 commits into from Aug 13, 2013
Merged

Brew install #431

merged 5 commits into from Aug 13, 2013

Conversation

jburel
Copy link
Member

@jburel jburel commented Aug 1, 2013

Few adjustments to the doc after a fresh install on 10.8

@hflynn
Copy link
Member

hflynn commented Aug 2, 2013

The actual content looks fine but I wonder about the formatting - this subsection now consists almost entirely of note boxes which makes it difficult to parse. Maybe you could replace the two notes with two bullet point sections?

@jburel
Copy link
Member Author

jburel commented Aug 2, 2013

@hflynn: good point.

@jburel
Copy link
Member Author

jburel commented Aug 2, 2013

@hflynn:

  • Moved the comment about ice34 installation to another section.
  • Removed the notes.
  • Added line to edit variable in .profile

@hflynn
Copy link
Member

hflynn commented Aug 2, 2013

Good to merge now.

@sbesson
Copy link
Member

sbesson commented Aug 5, 2013

A quick comment, it may be easier to document the following signatures to install OMERO with Ice 3.4

brew install omero --with-ice34

and to install the dependencies

brew install `brew deps omero --with-ice34` 

Although the latter is untested (i.e. assuming the options are properly propagated with brew deps)

@bpindelski
Copy link

I've tested the install docs from this PR and they make sense. Installed Ice 3.4 through homebrew with the caveat that I wasn't able to install when a virtualenv was active. python_deps.sh worked ok inside a virtualenv. I skipped the Postgres homebrew installation - I found the .dmg file supplied by PostgreSQL more user-friendly and quicker to install.

@sbesson
Copy link
Member

sbesson commented Aug 9, 2013

Thanks Blazej, regarding your comment, Homebrew does not work with virtualenv at installation time. Is it worth adding a line about this on this page with a link to the relevant documentation section
https://github.com/mxcl/homebrew/wiki/Homebrew-and-Python#virtualenv ?

@bpindelski
Copy link

@sbesson If it won't add noise in the documentation, then I don't see why not. It's quite relevant when doing brew install zeroc-ice34. In general I ended up using the OS X Python, on top of which I added pip (sudo easy_install pip) and finally did sudo pip install virtualenv and sudo pip install virtualenvwrapper. Then all the Omero deps I placed in the virtualenv. Seems to work fine for me. I agree that it would be nicer if we could just use a centralised Python (e.g. only homebrew Python), but that doesn't seem like a stable solution.

@sbesson
Copy link
Member

sbesson commented Aug 12, 2013

@bpindelski: makes sense. This discussion is no blocker for this PR. Do you want to open a ticket to review the Python part of the installation and assign me as the owner?

@bpindelski
Copy link

@joshmoore
Copy link
Member

Any final comments?

@sbesson
Copy link
Member

sbesson commented Aug 13, 2013

Merge :)

joshmoore added a commit that referenced this pull request Aug 13, 2013
@joshmoore joshmoore merged commit a9fa290 into ome:dev_4_4 Aug 13, 2013
@jburel jburel mentioned this pull request Aug 13, 2013
@jburel
Copy link
Member Author

jburel commented Sep 9, 2013

--rebased-to #439

@jburel jburel deleted the brew-install branch December 8, 2016 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants