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

Improved install docs (Vagrant, Homebrew, MAMP) #4064

Merged
merged 4 commits into from Apr 19, 2015

Conversation

chillu
Copy link
Member

@chillu chillu commented Apr 8, 2015

Modified some offline instructions from @halkyon to get Homebrew going, but went the custom PHP binary route rather than mucking around with PECL. More details in commit messages.

@camfindlay Are you OK with adding the Bitnami reference?
@dhensby Are you OK with linking to your/BetterBrief's Vagrant installer?


We're installing PHP 5.5 here, with the required `mcrypt` module:

brew install php55 php55-mcrypt
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't you have to setup some custom tap to use the php packages with homebrew?

Copy link
Contributor

Choose a reason for hiding this comment

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

We run the following before installing php with brew:

brew cleanup
brew doctor
brew update
brew tap homebrew/dupes
brew tap josegonzalez/homebrew-php
brew tap --repair

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point, done

@chillu chillu force-pushed the pulls/osx-install-docs branch 4 times, most recently from 0b32af6 to 7e4b845 Compare April 8, 2015 22:43
@chillu
Copy link
Member Author

chillu commented Apr 8, 2015

@stevie-mayhew Thanks for the feedback, I've linked to the troubleshooting docs for Homebrew. Good point on xcode-select, I've checked out a few tutorials around the web and appears that you're right. I've removed the download link.

@halkyon
Copy link
Contributor

halkyon commented Apr 8, 2015

My beautiful IIS 6 docs are being removed... ;-)
Looks good to me.

@dhensby
Copy link
Contributor

dhensby commented Apr 9, 2015

@chillu happy to have the vagrant repo linked to :)

I would go into more detail about why vagrant is so awesome (portable dev environments, very few local deps, shareable and identical environments for your team, "like live" environments)

@chillu
Copy link
Member Author

chillu commented Apr 9, 2015

@dhensby Awesome! Want to make a rewording suggestion for Vagrant? Is anything blocking the merge still?

 * Removed the `php.ini` configuration bits from both MAMP and Homebrew instructions, both actually come with reasonable defaults now (high `memory_limit`, `error_reporting` on full, `date.timezone` set).
 * Removed chapter about package install, we should favour composer now
 * Compressed the install wizard instructions
Mention composer there rather than just the package download.
Not linked from anywhere (other than the menu).
IIS 6.0 is part of Windows 2003, which has been EOLed as far as I can tell:
https://support.microsoft.com/en-us/lifecycle?p1=3198
halkyon pushed a commit that referenced this pull request Apr 19, 2015
Improved install docs (Vagrant, Homebrew, MAMP)
@halkyon halkyon merged commit 78e01c0 into silverstripe:3.1 Apr 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants