From: Mike Arpaia <mike@arpaia.co>
Date: Fri, 26 Jun 2015 18:37:16 +0000 (-0700)
Subject: Remove double-conversion as a source dep in OS X script
X-Git-Tag: v0.48.0~4
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=b75ef0a0af48766298ebcc946dd31fe0da5161e3;p=folly.git

Remove double-conversion as a source dep in OS X script

Summary: double-conversion was recently added to homebrew, making it the last
folly dependency to do so. this diff modifies the existing OS X
bootstrap tool to account for that. I also added an rm for the gtest
wget, so there wasn't a leftover zip file in your repo. alternatively,
we could remove that and add it to the gitignore.

Reviewed By: @fugalh

Differential Revision: D2196000

Pulled By: @sgolemon
---

diff --git a/.gitignore b/.gitignore
index 89d8518f..26bdeae2 100644
--- a/.gitignore
+++ b/.gitignore
@@ -13,7 +13,7 @@ aclocal.m4
 autom4te.cache
 build-aux
 libtool
-folly/test/gtest-1.6.0
+folly/test/gtest-1.*
 folly/folly-config.h
 folly/test/*_benchmark
 folly/test/*_test
diff --git a/folly/bootstrap-osx-homebrew.sh b/folly/bootstrap-osx-homebrew.sh
index 56e76bf5..688ce4dd 100755
--- a/folly/bootstrap-osx-homebrew.sh
+++ b/folly/bootstrap-osx-homebrew.sh
@@ -10,27 +10,13 @@ brewget() {
 }
 
 # tool dependencies: autotools and scons (for double-conversion)
-brewget autoconf automake libtool scons
+brewget autoconf automake libtool
 
 # dependencies
-brewget glog gflags boost libevent
-
-# Install the double-conversion library.
-# NB their install target installs the libs but not the headers, hence the
-# CPPFLAGS
-test -d double-conversion ||
-    git clone https://github.com/google/double-conversion.git
-pushd double-conversion
-make
-# fool libtool into using static linkage
-# (this won't work if you've already installed libdouble-conversion into a
-# default search path)
-rm -f libdouble-conversion*dylib
-DOUBLE_CONVERSION_HOME=$(pwd)
-popd
+brewget glog gflags boost libevent double-conversion
 
 autoreconf -i
-./configure CPPFLAGS=-I"$DOUBLE_CONVERSION_HOME" LDFLAGS=-L"$DOUBLE_CONVERSION_HOME"
+./configure
 
 pushd test
 test -e gtest-1.7.0.zip || {