From aab21a96523d3ebebe0d71d13d99fd1fa321727e Mon Sep 17 00:00:00 2001 From: "trop[bot]" <37223003+trop[bot]@users.noreply.github.com> Date: Tue, 23 Apr 2024 11:24:26 -0400 Subject: [PATCH] build: fixup codespaces on-create (#41935) Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr --- .devcontainer/devcontainer.json | 3 ++- .devcontainer/on-create-command.sh | 25 ++++++++----------------- 2 files changed, 10 insertions(+), 18 deletions(-) diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json index 3743ea60ae..54189514f8 100644 --- a/.devcontainer/devcontainer.json +++ b/.devcontainer/devcontainer.json @@ -31,7 +31,8 @@ ] }, "vscode": { - "extensions": ["joeleinbinder.mojom-language", + "extensions": [ + "joeleinbinder.mojom-language", "rafaelmaiolla.diff", "surajbarkale.ninja", "ms-vscode.cpptools", diff --git a/.devcontainer/on-create-command.sh b/.devcontainer/on-create-command.sh index 4e9ffd6f23..e168a5f0a5 100755 --- a/.devcontainer/on-create-command.sh +++ b/.devcontainer/on-create-command.sh @@ -39,7 +39,6 @@ if [ ! -f $buildtools/configs/evm.testing.json ]; then write_config() { echo " { - \"goma\": \"$1\", \"root\": \"/workspaces/gclient\", \"remotes\": { \"electron\": { @@ -49,7 +48,7 @@ if [ ! -f $buildtools/configs/evm.testing.json ]; then \"gen\": { \"args\": [ \"import(\\\"//electron/build/args/testing.gn\\\")\", - \"import(\\\"/home/builduser/.electron_build_tools/third_party/goma.gn\\\")\" + \"use_remoteexec = true\" ], \"out\": \"Testing\" }, @@ -57,26 +56,18 @@ if [ ! -f $buildtools/configs/evm.testing.json ]; then \"CHROMIUM_BUILDTOOLS_PATH\": \"/workspaces/gclient/src/buildtools\", \"GIT_CACHE_PATH\": \"/workspaces/gclient/.git-cache\" }, - \"\$schema\": \"file:///home/builduser/.electron_build_tools/evm-config.schema.json\" + \"\$schema\": \"file:///home/builduser/.electron_build_tools/evm-config.schema.json\", + \"configValidationLevel\": \"strict\", + \"reclient\": \"$1\", + \"goma\": \"none\", + \"preserveXcode\": 5 } " >$buildtools/configs/evm.testing.json } - # Start out as cache only - write_config cache-only + write_config remote_exec - e use testing - - # Attempt to auth to the goma service via codespaces tokens - # if it works we can use the goma cluster - export NOTGOMA_CODESPACES_TOKEN=$GITHUB_TOKEN - if e d goma_auth login; then - echo "$GITHUB_USER has GOMA access - switching to cluster mode" - write_config cluster - fi + e use testing else echo "build-tools testing config already exists" - - # Re-auth with the goma cluster regardless. - NOTGOMA_CODESPACES_TOKEN=$GITHUB_TOKEN e d goma_auth login || true fi