This reverts commit fc3fb20e.

Reason for revert: Previous versions of this changed used %scriptdir% in
goma_ctl.bat and goma_auth.bat but never defined it. This eluded manual
testing because these scripts would work properly as long as they were
run from the depot_tools directory, but would fail when run from
elsewhere.

The fix to make this reland work can be found by comparing patchset 1 to
patchset 3.

Original change's description:
> Revert "Reland "use python3 for goma related scripts""
>
> This reverts commit 2a6dffaf.
>
> Reason for revert: with this change in depot_tools when I run "goma_ctl ensure_start" from cmd.exe I get
> "The system cannot find the path specified."
>
> Original change's description:
> > Reland "use python3 for goma related scripts"
> >
> > This reverts commit 59140d43.
> >
> > Reason for revert:
> > Make this works in git bash.
> >
> > Original change's description:
> > > Revert "use python3 for goma related scripts"
> > >
> > > This reverts commit 138ec7c5.
> > >
> > > Reason for revert:
> > > This might have caused the following issue:
> > > https://bugs.chromium.org/p/chromium/issues/detail?id=1221233
> > >
> > > Original change's description:
> > > > use python3 for goma related scripts
> > > >
> > > > This uses python3 like autoninja{,.bat}.
> > > > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja
> > > > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja.bat
> > > >
> > > > Change-Id: I4daf1892024358c5b7d88b1e32b15c82cb4411d2
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2968069
> > > > Commit-Queue: Dirk Pranke <dpranke@google.com>
> > > > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > > > Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
> > > > Reviewed-by: Dirk Pranke <dpranke@google.com>
> > >
> > > Change-Id: Ideb5004cf18c91c1404066622244a8527f5248f3
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2972023
> > > Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
> > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > Commit-Queue: Dirk Pranke <dpranke@google.com>
> > > Reviewed-by: Dirk Pranke <dpranke@google.com>
> >
> > Bug: 1221233
> > Change-Id: I815a774faa2e4fbca0f262667ea89709a09e146b
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2979560
> > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > Reviewed-by: Allen Bauer <kylixrd@chromium.org>
> > Reviewed-by: Dirk Pranke <dpranke@google.com>
> > Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
> > Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
>
> Bug: 1221233
> Change-Id: I5050b9199f3076e6d6f812b8bac01e8cd1dc53bf
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2992150
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Reviewed-by: Dirk Pranke <dpranke@google.com>
> Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

Bug: 1221233
Change-Id: I50412a5ea62f1607660cb7a7ab8851251c3dfe91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2992151Reviewed-by: Dirk Pranke <dpranke@google.com>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

This reverts commit 2a6dffaf.

Reason for revert: with this change in depot_tools when I run "goma_ctl ensure_start" from cmd.exe I get
"The system cannot find the path specified."

Original change's description:
> Reland "use python3 for goma related scripts"
>
> This reverts commit 59140d43.
>
> Reason for revert:
> Make this works in git bash.
>
> Original change's description:
> > Revert "use python3 for goma related scripts"
> >
> > This reverts commit 138ec7c5.
> >
> > Reason for revert:
> > This might have caused the following issue:
> > https://bugs.chromium.org/p/chromium/issues/detail?id=1221233
> >
> > Original change's description:
> > > use python3 for goma related scripts
> > >
> > > This uses python3 like autoninja{,.bat}.
> > > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja
> > > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja.bat
> > >
> > > Change-Id: I4daf1892024358c5b7d88b1e32b15c82cb4411d2
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2968069
> > > Commit-Queue: Dirk Pranke <dpranke@google.com>
> > > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > > Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
> > > Reviewed-by: Dirk Pranke <dpranke@google.com>
> >
> > Change-Id: Ideb5004cf18c91c1404066622244a8527f5248f3
> > No-Presubmit: true
> > No-Tree-Checks: true
> > No-Try: true
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2972023
> > Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > Commit-Queue: Dirk Pranke <dpranke@google.com>
> > Reviewed-by: Dirk Pranke <dpranke@google.com>
>
> Bug: 1221233
> Change-Id: I815a774faa2e4fbca0f262667ea89709a09e146b
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2979560
> Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> Reviewed-by: Allen Bauer <kylixrd@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@google.com>
> Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
> Commit-Queue: Takuto Ikuta <tikuta@chromium.org>

Bug: 1221233
Change-Id: I5050b9199f3076e6d6f812b8bac01e8cd1dc53bf
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2992150
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

This reverts commit 59140d43.

Reason for revert:
Make this works in git bash.

Original change's description:
> Revert "use python3 for goma related scripts"
>
> This reverts commit 138ec7c5.
>
> Reason for revert:
> This might have caused the following issue:
> https://bugs.chromium.org/p/chromium/issues/detail?id=1221233
>
> Original change's description:
> > use python3 for goma related scripts
> >
> > This uses python3 like autoninja{,.bat}.
> > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja
> > https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja.bat
> >
> > Change-Id: I4daf1892024358c5b7d88b1e32b15c82cb4411d2
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2968069
> > Commit-Queue: Dirk Pranke <dpranke@google.com>
> > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
> > Reviewed-by: Dirk Pranke <dpranke@google.com>
>
> Change-Id: Ideb5004cf18c91c1404066622244a8527f5248f3
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2972023
> Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Commit-Queue: Dirk Pranke <dpranke@google.com>
> Reviewed-by: Dirk Pranke <dpranke@google.com>

Bug: 1221233
Change-Id: I815a774faa2e4fbca0f262667ea89709a09e146b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2979560
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Allen Bauer <kylixrd@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>

This reverts commit 138ec7c5.

Reason for revert:
This might have caused the following issue:
https://bugs.chromium.org/p/chromium/issues/detail?id=1221233

Original change's description:
> use python3 for goma related scripts
>
> This uses python3 like autoninja{,.bat}.
> https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja
> https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja.bat
>
> Fixed: 1203012
> Change-Id: I4daf1892024358c5b7d88b1e32b15c82cb4411d2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2968069
> Commit-Queue: Dirk Pranke <dpranke@google.com>
> Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
> Reviewed-by: Dirk Pranke <dpranke@google.com>

Change-Id: Ideb5004cf18c91c1404066622244a8527f5248f3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2972023
Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>

This uses python3 like autoninja{,.bat}.
https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja
https://source.chromium.org/chromium/chromium/tools/depot_tools/+/main:autoninja.bat

Fixed: 1203012
Change-Id: I4daf1892024358c5b7d88b1e32b15c82cb4411d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2968069
Commit-Queue: Dirk Pranke <dpranke@google.com>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>

Add revert instructions to .bat and bash files which are the entry
points to various infra tools deployed via CIPD.

Bug: 1206818
Change-Id: Ib0045aea86f29d0f7e3a43c6ff314c4b7aa74171
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2892387
Auto-Submit: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>

This was switched from python to vpython in
https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1962721
for a Windows module. Since vpython doesn't yet work on arm macs, switch
this back to vanilla python on non-win for now.

Bug: 1103236
Change-Id: Id083ad88d8474e283fb9f48d91e343b66aa67e51
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2842346
Auto-Submit: Nico Weber <thakis@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>

goma_ctl.py recommends that folks run `pip install pywin32`, which will
work up until python in depot_tools is updated, at which point it will
break again.

Running with vpython means that the vpython spec which ships with goma
will correctly take effect.

R=brucedawson@chromium.org, tikuta@chromium.org

Bug: 1033106
Change-Id: Icafadf381df782ab30d6cc156419dd7ed424d2bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1962721
Auto-Submit: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>

This reverts commit a0aed87f.

Reason for revert: install goma client without update_hook

update_hook would disrupt current users, so start without update_hook,
which means goma cient in depot_tools user might need to
restart compiler_proxy manually when updated.
 https://docs.google.com/document/d/1pnwfkU6Rd9dRtQC0sg2vATmyRbkYWhnNUTD5k1PddC0/edit#

Original change's description:
> Revert "put goma client in depot_tools"
>
> This reverts commit 77780358.
>
> Reason for revert: AttributeError: 'GomaEnvPosix' object has no attribute 'RestartCompilerProxy'
>
> Original change's description:
> > put goma client in depot_tools
> >
> > install goma client cipd package in depot_tools.
> >
> > should not use $MYPATH/goma_ctl in cipd_bin_setup
> > since $MYPATH/goma_ctl uses cipd_bin_setup in itself,
> > so causing recursive calls.
> > invoke python to run .cipd/goma_ctl.py in cipd_bin_setup
> > instead.
> >
> > Bug: b/77663154
> > Change-Id: I9f82c766a886a2acfb899e3594e5f05a7b7bc75a
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1866350
> > Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> > Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> > Commit-Queue: Fumitoshi Ukai <ukai@chromium.org>
>
> TBR=sque@chromium.org,ukai@chromium.org,yyanagisawa@google.com,vadimsh@chromium.org,dpranke@chromium.org,tikuta@chromium.org,ehmaldonado@chromium.org,yekuang@google.com
>
> Change-Id: Ie050dfb524dd885634c31be829d733613e80aece
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: b/77663154
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1872129
> Reviewed-by: Fumitoshi Ukai <ukai@chromium.org>
> Commit-Queue: Fumitoshi Ukai <ukai@chromium.org>

TBR=sque@chromium.org,ukai@chromium.org,yyanagisawa@google.com,vadimsh@chromium.org,dpranke@chromium.org,tikuta@chromium.org,ehmaldonado@chromium.org,yekuang@google.com


Bug: b/77663154
Change-Id: I8bb51631e4418ff63953099814bdb464128eb279
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1875982Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Fumitoshi Ukai <ukai@chromium.org>

on non-linux systems bash is not the default shell (by default) and
lives in a completely different path

Change-Id: I6580c776dda1b60b3d5c5f5f1ce4d6694ac45fe9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1605902Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>

It knows how to interpret configuration files written in Starlark-based
DSL and produce a bunch of protobuf messages as a result.

Will be used to generate various infra configs. In particular, will be
used by infra developers (to update configs), by PRESUBMIT.py on dev
machines (to verify configs before uploading CLs) and by bots (to verify
configs before submitting CLs).

R=tandrii@chromium.org
BUG=833946

Change-Id: Iceec7d808ce180f7d4a341fab8b5ce11933c2a6b
Reviewed-on: https://chromium-review.googlesource.com/c/1444499
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

Have the "update_depot_tools" script also do a CIPD tool sync. This will
ensure that users and systems have access to tooling at sync-time rather
than just-in-time loading them at execution time.

Update the tool boostraps to suppress any sort of syncing logs, if it
does happen. This will ensure that users who execute the tools don't see
unexpected output.

BUG=chromium:748651
TEST=local
  - Tested on Mac and Windows.

Change-Id: I1aad897d885a07beeac40a372a658681720efd2a
Reviewed-on: https://chromium-review.googlesource.com/591229
Commit-Queue: Daniel Jacques <dnj@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

This reverts commit 4d92fe43.

Reason for revert: This moved the logging from invoking `led` or `vpython` to whenever someone would run something that would end up invoking `update_depot_tools`. It's good that we are calling this when we run update_depot_tools, but we probably should've suppressed the logging there in at least the success case, because now things are even more confusing. See crbug.com/748651.

Original change's description:
> [bootstraps] Sync at gclient, suppress output.
> 
> Have the "update_depot_tools" script also do a CIPD tool sync. This will
> ensure that users and systems have access to tooling at sync-time rather
> than just-in-time loading them at execution time.
> 
> Update the tool boostraps to suppress any sort of syncing logic, if it
> does happen. This will ensure that users who execute the tools don't se
> unexpected output.
> 
> BUG=None
> TEST=local
>   - Tested on Mac and Windows.
> 
> R=​dpranke@chromium.org, iannucci@chromium.org
> 
> Change-Id: I8efce8c73cc4e82ffdf5067ba9b917119a81e843
> Reviewed-on: https://chromium-review.googlesource.com/581494
> Commit-Queue: Daniel Jacques <dnj@chromium.org>
> Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

TBR=iannucci@chromium.org,dpranke@chromium.org,dnj@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: None
Change-Id: I2485c9dd2e48a8dbdeebfff5da9d4c708e0edcb7
Reviewed-on: https://chromium-review.googlesource.com/585867Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>

Have the "update_depot_tools" script also do a CIPD tool sync. This will
ensure that users and systems have access to tooling at sync-time rather
than just-in-time loading them at execution time.

Update the tool boostraps to suppress any sort of syncing logic, if it
does happen. This will ensure that users who execute the tools don't se
unexpected output.

BUG=None
TEST=local
  - Tested on Mac and Windows.

R=dpranke@chromium.org, iannucci@chromium.org

Change-Id: I8efce8c73cc4e82ffdf5067ba9b917119a81e843
Reviewed-on: https://chromium-review.googlesource.com/581494
Commit-Queue: Daniel Jacques <dnj@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

R=nodir@chromium.org

Bug: 662654
Change-Id: I18898cb426ccbb895a82410dc61910a8b8e48f14
Reviewed-on: https://chromium-review.googlesource.com/517664
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>

BUG=None
TEST=None
R=sergeyberezin@chromium.org

Change-Id: I2bb22ad002b82767c6a7efb68412f9c1dbf57c27
Reviewed-on: https://chromium-review.googlesource.com/502228Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>

BUG=717208
R=dnj@chromium.org

Change-Id: If937a382be9aa4d8eb5f957386e8b1b28cc1c3ac
Reviewed-on: https://chromium-review.googlesource.com/492086Reviewed-by: Daniel Jacques <dnj@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org>