Tarantool development patches archive
 help / color / mirror / Atom feed
From: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
To: Oleg Piskunov <o.piskunov@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: [Tarantool-patches] [PATCH v4 1/2] build: tune OSX environment
Date: Tue, 21 Jan 2020 19:02:57 +0300	[thread overview]
Message-ID: <013e27e5dd6554a22f6e02b03a2383c5d2102000.1579621530.git.avtikhon@tarantool.org> (raw)
In-Reply-To: <cover.1579621530.git.avtikhon@tarantool.org>
In-Reply-To: <cover.1579621530.git.avtikhon@tarantool.org>

Added homebrew installation routine as it was suggested in its
instructions. Added installation of the cmake tool. Added upgrade
of the OSX packages to avoid of fails on already existed packages
with the previous versions.

Added reinstallation of the python2 with force option to be able
to install it to /usr/local/ path to make pip install there too.
---
 .travis.mk | 37 +++++++++++++++++++++++++++++--------
 1 file changed, 29 insertions(+), 8 deletions(-)

diff --git a/.travis.mk b/.travis.mk
index 42969ff56..def513713 100644
--- a/.travis.mk
+++ b/.travis.mk
@@ -127,17 +127,38 @@ test_asan_debian: deps_debian deps_buster_clang_8 test_asan_debian_no_deps
 # OSX #
 #######
 
+# WARNING: installing pip it checks that python2 installed in /usr/local/
+# path while python2 could be installed in /usr/bin path which will be
+# the cause that pip installation will be done at /User/tarantool/ path,
+# to avoid of it python2 must be reinstalled with force option to be able
+# to install it to /usr/local/ path and pip will be installed there too.
+OSX_PKGS=openssl readline curl icu4c libiconv zlib autoconf automake libtool cmake python2
+
 deps_osx:
-	brew update
-	brew install openssl readline curl icu4c libiconv zlib autoconf automake libtool --force
-	python2 -V || brew install python2 --force
-	curl --silent --show-error --retry 5 https://bootstrap.pypa.io/get-pip.py >get-pip.py
-	python get-pip.py --user
-	pip install --user --force-reinstall -r test-run/requirements.txt
+	# install brew using command from Homebrew repository instructions:
+	#   https://github.com/Homebrew/install
+	# NOTE: 'echo' command below is required since brew installation
+	# script obliges the one to enter a newline for confirming the
+	# installation via Ruby script.
+	#
+	# try to install the packages either upgrade it to avoid of fails
+	# if the package already exists with the previous version
+	#
+	# set PATH to /usr/local/bin as the main to be able to use brew
+	# installed python and tools
+	export PATH=/usr/local/bin:${PATH} ; \
+		brew update || echo | /usr/bin/ruby -e \
+			"$$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)" && \
+		brew install --force ${OSX_PKGS} || brew upgrade ${OSX_PKGS} && \
+		curl --silent --show-error --retry 5 https://bootstrap.pypa.io/get-pip.py >get-pip.py && \
+		python get-pip.py && \
+		pip install --force-reinstall -r test-run/requirements.txt
 
 build_osx:
-	cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_WERROR=ON ${CMAKE_EXTRA_PARAMS}
-	make -j
+	export PATH=/usr/local/bin:${PATH} ; \
+		cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo \
+			-DENABLE_WERROR=ON ${CMAKE_EXTRA_PARAMS} && \
+		make -j
 
 test_osx_no_deps: build_osx
 	# Limits: Increase the maximum number of open file descriptors on macOS:
-- 
2.17.1

  reply	other threads:[~2020-01-21 16:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-21 16:02 [Tarantool-patches] [PATCH v4 0/2] add Catalina OSX 10.15 to gitlab-ci Alexander V. Tikhonov
2020-01-21 16:02 ` Alexander V. Tikhonov [this message]
2020-01-21 16:21   ` [Tarantool-patches] [PATCH v4 1/2] build: tune OSX environment Oleg Piskunov
2020-01-21 16:02 ` [Tarantool-patches] [PATCH v4 2/2] gitlab-ci: add Catalina OSX 10.15 Alexander V. Tikhonov
2020-01-21 16:21   ` Oleg Piskunov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=013e27e5dd6554a22f6e02b03a2383c5d2102000.1579621530.git.avtikhon@tarantool.org \
    --to=avtikhon@tarantool.org \
    --cc=o.piskunov@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v4 1/2] build: tune OSX environment' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox