-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Any documentation for use of (or disabling of) reclient? #545
Comments
So I was able disable reclient by setting |
@alexciarlillo it's only within the last few weeks been enabled upstream, and we've only just gotten our deployment rolled out to maintainers courtesy of @MarshallOfSound. We'll let you know when it's more available for external contributrs but for now you should continue to use the goma cache. reclient usage also defaults to |
Thanks I may be reading this incorrectly but it looks as of #537 merged yesterday, specifically this change it may no longer be defaulting to |
I'm also looking for "What is the recommended configuration for non-maintainers?". Running {
"$schema": "file:///c%3A/Users/Administrator/.electron_build_tools/evm-config.schema.json",
"goma": "none",
"reclient": "none",
"root": "C:\\Users\\Administrator\\.electron_build_tools\\electron",
"remotes": {
"electron": {
"origin": "[email protected]:electron/electron.git",
"fork": "[email protected]:nikitakot/electron.git"
}
},
"gen": {
"args": [
"import(\"//electron/build/args/testing.gn\")"
],
"out": "Testing"
},
"env": {
"CHROMIUM_BUILDTOOLS_PATH": "C:\\Users\\Administrator\\.electron_build_tools\\electron\\src\\buildtools",
"GIT_CACHE_PATH": "C:\\Users\\Administrator\\.git_cache"
}
} |
@MarshallOfSound what are your thoughts here? |
@alexciarlillo Hi there, sorry if this is a dumb question but where exactly did you change this config? I am a newbie to electron development and whenever I run
I am just trying to get a local build working, not even sure what |
@SheikhSajid configs are in |
It's supposed to work for external users as well, in cache-only mode. Can someone provide details on what happens if you run the suggested |
@dsanders11 I don't get an "exception", but something akin to a |
same experience |
This is what I am getting as well |
any updates? should/can non maintainers continue to use goma? during the recent builds the cache-hit dropped significantly or switch to reclient is preferred? if reclient is preferred is there a way to authenticate in scripting environment without opening a browser window? thanks. |
Hi all - we think this may be resolved now, can you try authenticating again? Non-maintainers should get cache-only access. |
@dsanders11 it seems to work fine now. |
@BurningEnlightenment were you able to build? the Running "C:\Users\Administrator\.electron_build_tools\third_party\depot_tools\gn.bat gen out/Release --args=import("//electron/build/args/release.gn") use_remoteexec = true target_cpu="x86"" in C:\Users\Administrator\electron-reclient\src
reclient config file "../../buildtools/reclient_cfgs/chromium-browser-clang/rewrapper_windows.cfg" doesn't exist, you may need to set
"download_remoteexec_cfg" in .gclient like
`\``
solutions = [
{
"name" : "src",
# ...
"custom_vars" : {
"download_remoteexec_cfg": True,
},
},
]
`\``
and re-run `gclient sync`.
Or you may not set appropriate `rbe_cfg_dir` value in args.gn.
See
https://chromium.googlesource.com/chromium/src/+/main/docs/linux/build_instructions.md#use-reclient
for more details.
ERROR at //build/toolchain/rbe.gni:58:3: Script returned non-zero exit code.
exec_script(rebase_path("//build/toolchain/check_rewrapper_cfg.py"),
^----------
Current dir: C:/Users/Administrator/electron-reclient/src/out/Release/
Command: C:/Users/Administrator/.electron_build_tools/third_party/depot_tools/bootstrap-2@3_11_6_chromium_30_bin/python3/bin/python3.exe C:/Users/Administrator/electron-reclient/src/build/toolchain/check_rewrapper_cfg.py ../../buildtools/reclient_cfgs/chromium-browser-clang/rewrapper_windows.cfg
Returned 1.
See //build/config/BUILDCONFIG.gn:676:1: whence it was imported.
import("//build/toolchain/rbe.gni")
^---------------------------------
ERROR Error: Command failed: C:\Users\Administrator\.electron_build_tools\third_party\depot_tools\gn.bat gen out/Release --args=import("//electron/build/args/release.gn") use_remoteexec = true target_cpu="x86"
at checkExecSyncError (node:child_process:890:11)
at Object.execFileSync (node:child_process:926:15)
at Object.depotExecFileSync [as execFileSync] (C:\Users\Administrator\.electron_build_tools\src\utils\depot-tools.js:144:23)
at runGNGen (C:\Users\Administrator\.electron_build_tools\src\e-build.js:21:9)
at ensureGNGen (C:\Users\Administrator\.electron_build_tools\src\e-build.js:26:41)
at runNinja (C:\Users\Administrator\.electron_build_tools\src\e-build.js:61:3)
at Command.<anonymous> (C:\Users\Administrator\.electron_build_tools\src\e-build.js:139:9)
at Command.listener [as _actionHandler] (C:\Users\Administrator\.electron_build_tools\node_modules\commander\lib\command.js:480:17)
at C:\Users\Administrator\.electron_build_tools\node_modules\commander\lib\command.js:1234:65
node:internal/errors:865
const err = new Error(message);
^
Error: Command failed: e build electron:dist
at __node_internal_genericNodeError (node:internal/errors:865:15)
at checkExecSyncError (node:child_process:890:11)
at Object.execSync (node:child_process:962:15)
at build (c:\Users\Administrator\code\electron-build-tools\src\shared\build_tools.ts:88:6)
at <anonymous> (c:\Users\Administrator\code\electron-build-tools\src\build-electron.ts:45:1)
at ModuleJob.run (node:internal/modules/esm/module_job:195:25)
at async ModuleLoader.import (node:internal/modules/esm/loader:336:24)
at async loadESM (node:internal/process/esm_loader:34:7)
at async handleMainPromise (node:internal/modules/run_main:106:12) {
status: 1,
signal: null,
output: [ null, null, null ],
pid: 16164,
stdout: null,
stderr: null
}
Node.js v18.19.0 |
@nikitakot I've bootstrapped a new configuration as I wanted to move the directory somewhere else anyways. I didn't encounter any problems (though the toolchain download progressed very weirdly). |
@nikitakot, an |
@dsanders11 I've initialized a new config, tried |
FTR I built the current |
@nikitakot, x86 Windows is not a supported build platform. You can build x86 binaries with x64 Windows, but you can't build on x86 Windows. There's no upstream support in Chromium, so it's probably failing to find some needed executables. |
@dsanders11 Yes sorry I meant that I'm building Windows v28.2.3 target cpu x86, my machine is x64. With |
@dsanders11 maybe it is worth mentioning that I was trying to build 28.2.3 "upstream" tag which I "copied" into my private fork? this is what i mean: e init // ...some args
e sync
cd ... // to electron src folder
git checkout tags/v28.2.3 -b v28.2.3-my-branch
git push --set-upstream fork v28.2.3-my-branch
e sync
e build // doesn't work, reclient fails Can it be the issue? Does reclient expect some specific git history or a public repo? My fork is private. |
@nikitakot, sorry, I latched on to the x86 Windows and missed that you said you're trying to build v28. Support for Reclient is only on |
It's been a few months since I did an electron build and I just pulled down the latest build-tools this evening. I am now getting an authentication error related to RBE. According to this it seems like even Cache-Only access is limited to Electron maintainers? If this is not actually the case I would love to know how to properly configure Cache-Only access, or otherwise how to correctly disable use of reclient.
Thanks
The text was updated successfully, but these errors were encountered: