Skip to content

Commit

Permalink
Remove brew libomp from binary mac machines (#20228)
Browse files Browse the repository at this point in the history
Summary:
Resolves #20030
Pull Request resolved: #20228

Differential Revision: D15246258

Pulled By: pjh5

fbshipit-source-id: f57033af74b678566be02cdf5700b5ae6e154d4a
  • Loading branch information
pjh5 authored and facebook-github-bot committed May 8, 2019
1 parent eecf52b commit 9c62280
Show file tree
Hide file tree
Showing 4 changed files with 38 additions and 10 deletions.
24 changes: 19 additions & 5 deletions .circleci/config.yml
Expand Up @@ -281,8 +281,6 @@ macos_brew_update: &macos_brew_update
no_output_timeout: "1h"
command: |
set -ex
pwd
ls -lah
# moreutils installs a `parallel` executable by default, which conflicts
# with the executable from the GNU `parallel`, so we must unlink GNU
# `parallel` first, and relink it afterwards
Expand All @@ -293,6 +291,22 @@ macos_brew_update: &macos_brew_update
brew install expect
brew install libomp
# In version 2.1 and above we could make this a command and pass a parameter to
# it, but in this version there is no way to pass a parameter to a step
binary_macos_brew_update: &binary_macos_brew_update
name: Brew update and install moreutils and expect
no_output_timeout: "1h"
command: |
set -ex
# moreutils installs a `parallel` executable by default, which conflicts
# with the executable from the GNU `parallel`, so we must unlink GNU
# `parallel` first, and relink it afterwards
brew update
brew unlink parallel
brew install moreutils
brew link parallel --overwrite
brew install expect
##############################################################################
# Linux build defaults
Expand Down Expand Up @@ -770,7 +784,7 @@ binary_mac_build: &binary_mac_build
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
<<: *binary_install_miniconda

Expand Down Expand Up @@ -805,7 +819,7 @@ binary_mac_upload: &binary_mac_upload
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
<<: *binary_install_miniconda
- attach_workspace:
Expand Down Expand Up @@ -863,7 +877,7 @@ smoke_mac_test: &smoke_mac_test
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
name: Build
no_output_timeout: "1h"
Expand Down
18 changes: 16 additions & 2 deletions .circleci/verbatim-sources/header-section.yml
Expand Up @@ -281,8 +281,6 @@ macos_brew_update: &macos_brew_update
no_output_timeout: "1h"
command: |
set -ex
pwd
ls -lah
# moreutils installs a `parallel` executable by default, which conflicts
# with the executable from the GNU `parallel`, so we must unlink GNU
# `parallel` first, and relink it afterwards
Expand All @@ -292,3 +290,19 @@ macos_brew_update: &macos_brew_update
brew link parallel --overwrite
brew install expect
brew install libomp
# In version 2.1 and above we could make this a command and pass a parameter to
# it, but in this version there is no way to pass a parameter to a step
binary_macos_brew_update: &binary_macos_brew_update
name: Brew update and install moreutils and expect
no_output_timeout: "1h"
command: |
set -ex
# moreutils installs a `parallel` executable by default, which conflicts
# with the executable from the GNU `parallel`, so we must unlink GNU
# `parallel` first, and relink it afterwards
brew update
brew unlink parallel
brew install moreutils
brew link parallel --overwrite
brew install expect
4 changes: 2 additions & 2 deletions .circleci/verbatim-sources/macos-binary-build-defaults.yml
Expand Up @@ -12,7 +12,7 @@ binary_mac_build: &binary_mac_build
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
<<: *binary_install_miniconda

Expand Down Expand Up @@ -47,7 +47,7 @@ binary_mac_upload: &binary_mac_upload
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
<<: *binary_install_miniconda
- attach_workspace:
Expand Down
Expand Up @@ -44,7 +44,7 @@ smoke_mac_test: &smoke_mac_test
- run:
<<: *binary_populate_env
- run:
<<: *macos_brew_update
<<: *binary_macos_brew_update
- run:
name: Build
no_output_timeout: "1h"
Expand Down

0 comments on commit 9c62280

Please sign in to comment.