Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upUpdate README for Homebrew. #3991
Conversation
hoppipolla-critic-bot
commented
Nov 15, 2014
|
Critic review: https://critic.hoppipolla.co.uk/r/3174 This is an external review system which you may optionally use for the code review of your pull request. In order to help critic track your changes, please do not make in-place history rewrites (e.g. via |
|
There's a similar PR in #3895, but we actually should just remove glfw3 from the prerequisites entirely since it is no longer needed. Could you do that instead? |
|
@jdm Yup. I pulled it from the other parts of the README.md too. |
|
Thanks! |
|
Hmm, looks like this need a rebase. Sorry. |
|
Should be better. |
|
Still can't merge for some reason :/ |
|
The reason being that the commit now includes a bunch of sniffer-related code in net/. |
|
Oy, yeah, what the heck. |
glfw3 is in the `homebrew/homebrew-versions` repository.
|
@jdm Sorry about that. Should be happy now. Oh the folly of |
This comment has been minimized.
This comment has been minimized.
jdm
commented on 3b2fcb0
Nov 16, 2014
|
r+ |
This comment has been minimized.
This comment has been minimized.
jdm
replied
Nov 17, 2014
|
@bors: retry |
This comment has been minimized.
This comment has been minimized.
|
saw approval from jdm |
This comment has been minimized.
This comment has been minimized.
|
merging Hoverbear/servo/patch-1 = 3b2fcb0 into auto |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
some tests failed: |
This comment has been minimized.
This comment has been minimized.
|
saw approval from jdm |
This comment has been minimized.
This comment has been minimized.
|
merging Hoverbear/servo/patch-1 = 3b2fcb0 into auto |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
all tests pass: |
This comment has been minimized.
This comment has been minimized.
|
fast-forwarding master to auto = b9dcaa3 |
glfw3 is in the `homebrew/versions` keg. ```bash ➜ ~ brew install glfw3 Error: No available formula for glfw3 Searching formulae... Searching taps... homebrew/versions/glfw3 ``` Simplest fix: ```bash ➜ ~ brew install homebrew/versions/glfw3 Cloning into '/usr/local/Library/Taps/homebrew/homebrew-versions'... remote: Counting objects: 2495, done. remote: Total 2495 (delta 0), reused 0 (delta 0) Receiving objects: 100% (2495/2495), 820.05 KiB | 462.00 KiB/s, done. Resolving deltas: 100% (1433/1433), done. Checking connectivity... done. Tapped 161 formulae ==> Installing glfw3 from homebrew/homebrew-versions ==> Downloading https://downloads.sourceforge.net/project/glfw/glfw/3.0.4/glfw-3.0.4.tar.bz2 Already downloaded: /Library/Caches/Homebrew/glfw3-3.0.4.tar.bz2 ==> Patching patching file CMakeLists.txt Hunk #1 succeeded at 370 with fuzz 1 (offset 13 lines). ==> cmake -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/glfw3/3.0.4 -DCMAKE_BUILD_TYPE=None -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_VERBOSE_MAKEFILE=ON -Wno-dev -DGLFW_USE_CHDIR=TRUE -DGLFW_USE_MENUBAR=TRUE -DBUIL ==> make install🍺 /usr/local/Cellar/glfw3/3.0.4: 11 files, 220K, built in 9 seconds ``` Users who have `homebrew/versions` already tapped won't notice this, since after tapping `homebrew/versions` keg shouldn't notice, since after tapping, the library is available using just ` brew install glfw3`. ```bash ➜ ~ brew tap homebrew/versions ➜ ~ brew install glfw3 ==> Installing glfw3 from homebrew/homebrew-versions ==> Downloading https://downloads.sourceforge.net/project/glfw/glfw/3.0.4/glfw-3.0.4.tar.bz2 Already downloaded: /Library/Caches/Homebrew/glfw3-3.0.4.tar.bz2 ==> Patching patching file CMakeLists.txt Hunk #1 succeeded at 370 with fuzz 1 (offset 13 lines). ==> cmake -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/glfw3/3.0.4 -DCMAKE_BUILD_TYPE=None -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_VERBOSE_MAKEFILE=ON -Wno-dev -DGLFW_USE_CHDIR=TRUE -DGLFW_USE_MENUBAR=TRUE -DBUIL ==> make install🍺 /usr/local/Cellar/glfw3/3.0.4: 11 files, 220K, built in 9 seconds ```
glfw3 is in the `homebrew/versions` keg. ```bash ➜ ~ brew install glfw3 Error: No available formula for glfw3 Searching formulae... Searching taps... homebrew/versions/glfw3 ``` Simplest fix: ```bash ➜ ~ brew install homebrew/versions/glfw3 Cloning into '/usr/local/Library/Taps/homebrew/homebrew-versions'... remote: Counting objects: 2495, done. remote: Total 2495 (delta 0), reused 0 (delta 0) Receiving objects: 100% (2495/2495), 820.05 KiB | 462.00 KiB/s, done. Resolving deltas: 100% (1433/1433), done. Checking connectivity... done. Tapped 161 formulae ==> Installing glfw3 from homebrew/homebrew-versions ==> Downloading https://downloads.sourceforge.net/project/glfw/glfw/3.0.4/glfw-3.0.4.tar.bz2 Already downloaded: /Library/Caches/Homebrew/glfw3-3.0.4.tar.bz2 ==> Patching patching file CMakeLists.txt Hunk #1 succeeded at 370 with fuzz 1 (offset 13 lines). ==> cmake -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/glfw3/3.0.4 -DCMAKE_BUILD_TYPE=None -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_VERBOSE_MAKEFILE=ON -Wno-dev -DGLFW_USE_CHDIR=TRUE -DGLFW_USE_MENUBAR=TRUE -DBUIL ==> make install🍺 /usr/local/Cellar/glfw3/3.0.4: 11 files, 220K, built in 9 seconds ``` Users who have `homebrew/versions` already tapped won't notice this, since after tapping `homebrew/versions` keg shouldn't notice, since after tapping, the library is available using just ` brew install glfw3`. ```bash ➜ ~ brew tap homebrew/versions ➜ ~ brew install glfw3 ==> Installing glfw3 from homebrew/homebrew-versions ==> Downloading https://downloads.sourceforge.net/project/glfw/glfw/3.0.4/glfw-3.0.4.tar.bz2 Already downloaded: /Library/Caches/Homebrew/glfw3-3.0.4.tar.bz2 ==> Patching patching file CMakeLists.txt Hunk #1 succeeded at 370 with fuzz 1 (offset 13 lines). ==> cmake -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/glfw3/3.0.4 -DCMAKE_BUILD_TYPE=None -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_VERBOSE_MAKEFILE=ON -Wno-dev -DGLFW_USE_CHDIR=TRUE -DGLFW_USE_MENUBAR=TRUE -DBUIL ==> make install🍺 /usr/local/Cellar/glfw3/3.0.4: 11 files, 220K, built in 9 seconds ```
3b2fcb0
into
servo:master
| @@ -32,7 +28,7 @@ On Debian-based Linuxes: | |||
| sudo apt-get install curl freeglut3-dev \ | |||
| libfreetype6-dev libgl1-mesa-dri libglib2.0-dev xorg-dev \ | |||
| msttcorefonts gperf g++ cmake python-virtualenv \ | |||
| libssl-dev libglfw-dev libbz2-dev | |||
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
SimonSapin
Nov 20, 2014
Member
To clarify: bzip2 should not have been removed here and in other places.
|
@SimonSapin Sorry about that. I didn't notice that when I was attempting to rebase. :( |
|
Don’t worry, it’s nothing we can’t fix :) |
Hoverbear commentedNov 15, 2014
glfw3 is in the
homebrew/versionskeg.Simplest fix:
Users who have
homebrew/versionsalready tapped won't notice this, since after tappinghomebrew/versionskeg shouldn't notice, since after tapping, the library is available using justbrew install glfw3.