Use [CACHE]/cipd/goma for goma_dir in recipes/chromium_codesearch.py

Use [CACHE]/cipd/goma for goma_dir in recipes/chromium_codesearch.py

* use chromium.ensure_goma()

BUG=621828
Committed: https://chromium.googlesource.com/chromium/tools/build/+/bcdf12617db069e17f963d54e50d1afe44508911

[tikuta, 2016-09-12 09:29:36]

tikuta@chromium.org changed reviewers:
+ iannucci@chromium.org, shinyak@chromium.org, ukai@chromium.org,
yyanagisawa@chromium.org

[tikuta, 2016-09-12 09:29:37]

tikuta@chromium.org changed required reviewers:
+ iannucci@chromium.org

[shinyak, 2016-09-12 13:17:54]

lgtm

[iannucci, 2016-09-12 23:04:37]

lgtm

[Yoshisato Yanagisawa, 2016-09-13 01:48:09]

lgtm

[tikuta, 2016-09-13 02:42:50]

The CQ bit was checked by tikuta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 02:43:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ukai, 2016-09-13 02:43:25]

lgtm

[commit-bot: I haz the power, 2016-09-13 02:47:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 02:47:33]

Dry run: This issue passed the CQ dry run.

[tikuta, 2016-09-13 02:53:22]

The CQ bit was checked by tikuta@chromium.org

[commit-bot: I haz the power, 2016-09-13 02:53:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-13 02:59:22]

Description was changed from

==========
Use [CACHE]/cipd/goma for goma_dir in recipes/chromium_codesearch.py

* use chromium.ensure_goma()

BUG=621828
==========

to

==========
Use [CACHE]/cipd/goma for goma_dir in recipes/chromium_codesearch.py

* use chromium.ensure_goma()

BUG=621828

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/bcdf12617db069e17f96...
==========

[commit-bot: I haz the power, 2016-09-13 02:59:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/bcdf12617db069e17f96...

[ukai, 2016-09-13 03:52:58]

On 2016/09/13 02:59:23, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/tools/build/+/bcdf12617db069e17f96...

https://build.chromium.org/p/chromium.infra.cron/builders/ChromiumOS%20Codese...

FAILED: obj/breakpad/dump_syms/dump_syms.o 
/b/build/goma/gomacc ../../third_party/llvm-build/Release+Asserts/bin/clang++
-MMD -MF obj/breakpad/dump_syms/dump_syms.o.d -DHAVE_A_OUT_H
-DV8_DEPRECATION_WARNINGS -DENABLE_MDNS=1 -DENABLE_NOTIFICATIONS
-DENABLE_PEPPER_CDMS -DENABLE_PLUGINS=1 -DENABLE_PDF=1 -DENABLE_PRINTING=1
-DENABLE_BASIC_PRINTING=1 -DENABLE_PRINT_PREVIEW=1 -DENABLE_SPELLCHECK=1
-DUSE_UDEV -DUI_COMPOSITOR_IMAGE_TRANSPORT -DUSE_ASH=1 -DUSE_AURA=1
-DUSE_DEFAULT_RENDER_THEME=1 -DUSE_NSS_CERTS=1 -DUSE_OZONE=1 -DENABLE_WEBRTC=1
-DENABLE_EXTENSIONS=1 -DENABLE_TASK_MANAGER=1 -DENABLE_THEMES=1
-DENABLE_CAPTIVE_PORTAL_DETECTION=1 -DENABLE_SESSION_SERVICE=1
-DENABLE_APP_LIST=1 -DENABLE_SUPERVISED_USERS=1 -DENABLE_SERVICE_DISCOVERY=1
-DIMAGE_LOADER_EXTENSION=1 -DENABLE_WAYLAND_SERVER=1 -DFULL_SAFE_BROWSING
-DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DCHROMIUM_BUILD
-DENABLE_MEDIA_ROUTER=1 -DFIELDTRIAL_TESTING_ENABLED
-DCR_CLANG_REVISION=280836-1 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE
-D_LARGEFILE64_SOURCE -DCOMPONENT_BUILD -DOS_CHROMEOS -D_DEBUG
-DDYNAMIC_ANNOTATIONS_ENABLED=1 -DWTF_USE_DYNAMIC_ANNOTATIONS=1
-D_GLIBCXX_DEBUG=1 -I../../breakpad/src -I../.. -Igen -fno-strict-aliasing
-funwind-tables -fPIC -pipe -B../../third_party/binutils/Linux_x64/Release/bin
-fcolor-diagnostics
-fdebug-prefix-map=/b/build/slave/ChromiumOS_Codesearch/build/src=. -pthread
-m64 -march=x86-64 -O0 -fno-omit-frame-pointer -g1 -gsplit-dwarf
--sysroot=../../build/linux/ubuntu_precise_amd64-sysroot -fvisibility=hidden
-Xclang -load -Xclang
../../third_party/llvm-build/Release+Asserts/lib/libFindBadConstructs.so -Xclang
-add-plugin -Xclang find-bad-constructs -Xclang -plugin-arg-find-bad-constructs
-Xclang check-ipc -Wheader-hygiene -Wstring-conversion -Werror -Wall
-Wno-unused-variable -Wno-missing-field-initializers -Wno-unused-parameter
-Wno-c++11-narrowing -Wno-covered-switch-default -Wno-deprecated-register
-Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override
-Wno-shift-negative-value -Wno-undefined-var-template
-Wno-nonportable-include-path -Wno-address-of-packed-member
-fno-threadsafe-statics -fvisibility-inlines-hidden
-Wno-undefined-bool-conversion -Wno-tautological-undefined-compare -std=gnu++11
-fno-exceptions -Wno-deprecated -Wno-reserved-user-defined-literal -frtti -c
../../breakpad/src/tools/linux/dump_syms/dump_syms.cc -o
obj/breakpad/dump_syms/dump_syms.o
/bin/sh: 1: /b/build/goma/gomacc: not found

failed to configure goma dir in build rule?

[tikuta, 2016-09-13 03:55:54]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2335093002/ by tikuta@chromium.org.

The reason for reverting is:
https://build.chromium.org/p/chromium.infra.cron/builders/ChromiumOS%20Codese...

args for gn uses old goma_dir..