From 91609e249a79c913406ed6f3ed10a4661cad3528 Mon Sep 17 00:00:00 2001 From: Shinichiro Hamaji Date: Tue, 12 Apr 2016 18:14:28 +0900 Subject: [PATCH] Always use ensure_start to ensure goma's daemon runs Recent goma client automatically restarts when GOMA_* environment variable changes. http://b/25676777 was fixed 2 months ago, so there wouldn't be old goma client. Change-Id: I3b5419ce4bae4cf8180d8d0a53d2b4218d317fec --- core/goma.mk | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/core/goma.mk b/core/goma.mk index 982160b1d6..0d5f428b33 100644 --- a/core/goma.mk +++ b/core/goma.mk @@ -46,11 +46,7 @@ ifneq ($(filter-out false,$(USE_GOMA)),) # gomacc can start goma client's daemon process automatically, but # it is safer and faster to start up it beforehand. We run this as a # background process so this won't slow down the build. - # We use "ensure_start" command when the compiler_proxy is already - # running and uses GOMA_HERMETIC=error flag. The compiler_proxy will - # restart otherwise. - # TODO(hamaji): Remove this condition after http://b/25676777 is fixed. - $(shell ( if ( curl http://localhost:$$($(GOMA_CC) port)/flagz | grep GOMA_HERMETIC=error ); then cmd=ensure_start; else cmd=restart; fi; GOMA_HERMETIC=error $(goma_ctl) $${cmd} ) &> /dev/null &) + $(shell ( GOMA_HERMETIC=error $(goma_ctl) ensure_start ) &> /dev/null &) goma_ctl := goma_dir :=