From 7d49ce898afc8cdd7f1814fc6e89f3dc478e5dae Mon Sep 17 00:00:00 2001 From: Samuel Attard Date: Mon, 30 Nov 2020 12:29:36 -0800 Subject: [PATCH] build: use all-for-one goma (#26699) * Revert "Revert "build: use one-for-all goma (#26679)" (#26689)" This reverts commit 38ab829ea6f683dcba886590006cbceebb74fbfa. * build: ensure file descriptor limit is higher on macOS --- .circleci/config.yml | 10 +++++----- appveyor.yml | 3 ++- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index 0cb7b7b4a67f..099e2ae5c5a7 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -295,10 +295,10 @@ step-setup-goma-for-build: &step-setup-goma-for-build run: name: Setup Goma command: | - if [ "`uname`" == "Linux" ]; then - echo 'export NUMBER_OF_NINJA_PROCESSES=300' >> $BASH_ENV - else - echo 'export NUMBER_OF_NINJA_PROCESSES=25' >> $BASH_ENV + echo 'export NUMBER_OF_NINJA_PROCESSES=300' >> $BASH_ENV + if [ "`uname`" == "Darwin" ]; then + echo 'ulimit -n 10000' >> $BASH_ENV + echo 'sudo launchctl limit maxfiles 65536 200000' >> $BASH_ENV fi if [ ! -z "$RAW_GOMA_AUTH" ]; then echo $RAW_GOMA_AUTH > ~/.goma_oauth2_config @@ -307,7 +307,7 @@ step-setup-goma-for-build: &step-setup-goma-for-build cd build-tools npm install mkdir third_party - node -e "require('./src/utils/goma.js').downloadAndPrepare()" + node -e "require('./src/utils/goma.js').downloadAndPrepare({ gomaOneForAll: true })" node -e "require('./src/utils/goma.js').ensure()" echo 'export GN_GOMA_FILE='`node -e "console.log(require('./src/utils/goma.js').gnFilePath)"` >> $BASH_ENV echo 'export LOCAL_GOMA_DIR='`node -e "console.log(require('./src/utils/goma.js').dir)"` >> $BASH_ENV diff --git a/appveyor.yml b/appveyor.yml index d3c49d0115cd..1766f764f00a 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -139,7 +139,8 @@ build_script: - cd build-tools - npm install - mkdir third_party - - node -e "require('./src/utils/goma.js').downloadAndPrepare()" + - ps: >- + node -e "require('./src/utils/goma.js').downloadAndPrepare({ gomaOneForAll: true })" - ps: $env:GN_GOMA_FILE = node -e "console.log(require('./src/utils/goma.js').gnFilePath)" - ps: $env:LOCAL_GOMA_DIR = node -e "console.log(require('./src/utils/goma.js').dir)" - cd ..