diff --git a/jenkins-scripts/lib/_homebrew_base_setup.bash b/jenkins-scripts/lib/_homebrew_base_setup.bash index 35d2308fc..dfd607e91 100644 --- a/jenkins-scripts/lib/_homebrew_base_setup.bash +++ b/jenkins-scripts/lib/_homebrew_base_setup.bash @@ -16,7 +16,15 @@ fi git -C $(${BREW_BINARY} --repo) fsck export HOMEBREW_UPDATE_TO_TAG=1 -${BREW_BINARY} update + +# There might be a background process that blocks `brew update`, so we try to +# run it several times until it succeeds. +# See https://github.com/Homebrew/brew/issues/1155 +brew_update_retry_count=0 +until ${BREW_BINARY} update || (( brew_update_retry_count++ > 6 )) +do + sleep 10 +done # manually exclude a ruby warning that jenkins thinks is from clang # https://github.com/osrf/homebrew-simulation/issues/1343 ${BREW_BINARY} install ${BREW_BASE_DEPENDCIES} \ diff --git a/jenkins-scripts/lib/_homebrew_cleanup.bash b/jenkins-scripts/lib/_homebrew_cleanup.bash index 76cd0fb93..40d386055 100644 --- a/jenkins-scripts/lib/_homebrew_cleanup.bash +++ b/jenkins-scripts/lib/_homebrew_cleanup.bash @@ -39,16 +39,6 @@ git stash && git clean -d -f ${BREW_BINARY} audit cmake || restore_brew popd 2> /dev/null -# There might be a background process that blocks `brew update`, so we try to -# run it several times until it succeeds. -# See https://github.com/Homebrew/brew/issues/1155 -brew_update_retry_count=0 -until brew update || (( brew_update_retry_count++ > 6 )) -do - brew update - sleep 10 -done - # test-bot needs variables and does not work just with config not sure why export GIT_AUTHOR_NAME="OSRF Build Bot" export GIT_COMMITTER_NAME=${GIT_AUTHOR_NAME}