Update README for Yosemite #14
Conversation
I don't know if all this is needed for ElCapitan
| export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig | ||
| go get github.com/moovweb/gokogiri | ||
| go get github.com/opencontrol/fedramp-templater | ||
| ``` |
There was a problem hiding this comment.
Minor: Mind splitting "install the dependencies" from "install the tool"? Also, mind adding bash to the syntax blocks for highlighting?
|
Very helpful, thanks! Fixes #12. |
| brew install libxml2 | ||
| ln -s /usr/local/Cellar/libxml2/2.9.4/lib/pkgconfig/libxml-2.0.pc /usr/local/lib/pkgconfig/libxml-2.0.pc | ||
| export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig | ||
| go get github.com/moovweb/gokogiri |
There was a problem hiding this comment.
We use a fork, and this shouldn't be necessary anyway, since all the dependencies are vendored.
|
Note that with upgrading to Go 1.6.3 (or something else) on El Capitan, I needed to export PKG_CONFIG_PATH=/usr/local/opt/libxml2/lib/pkgconfig |
|
Regardless of what the instructions actually are, my preference would be to add them to https://github.com/jbowtie/gokogiri as a pull request and link there, so they stay up-to-date and are more centralized. |
|
I remember earlier trying to get this to work (on El Capitan) and it wouldn't (this is back in the beginning of this project). I just pulled down the latest code and the build just works. I even uninstalled libxml2 and it works. The tests work too. |
Per comments, move ugly details downstream.
Typo fix
|
@afeld @jcscottiii Yup, moved install questions over to the Gokogiri end... How now? |
|
Awesome, thanks! |
I don't know if all this is needed for ElCapitan