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.

Sign up for GitHub

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

Jump to bottom

Review the Homebrew installation page #264

Merged
merged 6 commits into from
Feb 25, 2013
Merged

Review the Homebrew installation page #264

merged 6 commits into from
Feb 25, 2013

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Feb 24, 2013

This PR updated the Homebrew installation page to reflect the changes brought:

Summary of changes:

  • modify the installation order as 1- Homebrew, 2- OMERO, 3- Postgresql, 4- Python deps
  • merge & update the OSX & Xcode requirements sections (10.5 no longer tested)
  • fix various text formatting markup
  • add a section about failing brew install gfortran under 32-bit Snow Leopard

@sbesson
Remove reference to 4.3.4 server
8e4c3a8
@sbesson
Move OMERO installation section before requirements
d4a13a5 
With the recent optional dependencies added to the OMERO formula, `brew install omero` already installs all the dependencies required to build the
server.
@sbesson
Update OMERO & Python dependencies installation using Homebrew
09c1ae1 
- Add `brew tap ome/alt` command.
- Move PostgreSQL and Python dependencies installation after OMERO
installation
- Add command to install OMERO dependencies only using Homebrew
@sbesson
Merge OS X and Xcode sections and remove OS X <=10.5
4757723
@sbesson
Add section about 32-bit gfortran and fix various text formatting
51dc26b
mtbc
mtbc reviewed Feb 25, 2013
View reviewed changes
sysadmins/unix/server-install-homebrew.txt
@@ -82,104 +77,97 @@ requirements for OMERO will be installed to ``/usr/local``.

If you are having issues with curl, see the the :ref:`install_homebrew_curl` section under :ref:`install_homebrew_common_issues`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

double "the" (not your fault!)

@mtbc
Copy link
Member

mtbc commented Feb 25, 2013

No showstoppers here, but probably still worth a look.

@mtbc
Copy link
Member

mtbc commented Feb 25, 2013

Good to merge.

@joshmoore
Copy link
Member

This looks good, @sbesson and will work out-of-the-box with your other PR. Before releasing 4.4.7, we may should consider though installing python_deps.sh under bin/ beside omero so that it becomes:

bin/omero_python_deps

rather than:

$ cd /usr/local
$ curl -fsSkL https://raw.github.com/openmicroscopy/openmicroscopy/dev_4_4/docs/install/python_deps.sh > python_deps.sh
$ source python_deps.sh

Merging.

joshmoore added a commit that referenced this pull request Feb 25, 2013
@joshmoore
Merge pull request #264 from sbesson/homebrew_deps
f8d8edc 
Review the Homebrew installation page
@joshmoore joshmoore merged commit f8d8edc into ome:dev_4_4 Feb 25, 2013
@sbesson sbesson mentioned this pull request Feb 25, 2013
Merged
@sbesson sbesson deleted the homebrew_deps branch February 25, 2013 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dev_4_4
Projects
None yet
Milestone
4.4.7
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@sbesson @mtbc @joshmoore