From 2ff30bec74e958f35510871df25c3bc88f3d7234 Mon Sep 17 00:00:00 2001 From: Andrew Dupont Date: Thu, 18 Dec 2025 15:44:59 -0800 Subject: [PATCH] =?UTF-8?q?Revert=20setting=20of=20`--force-process-config?= =?UTF-8?q?`=E2=80=A6?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit …now that we’re on Electron 30. --- src/command.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/command.js b/src/command.js index 6fb0bb49..68c19e6f 100644 --- a/src/command.js +++ b/src/command.js @@ -132,14 +132,13 @@ class Command { env.npm_config_disturl = config.getElectronUrl(); env.npm_config_arch = config.getElectronArch(); env.npm_config_target_arch = config.getElectronArch(); // for node-pre-gyp - env.npm_config_force_process_config = "true"; // for node-gyp } - // node-gyp >=8.4.0 needs --force-process-config=true set for older Electron. - // For more details, see: https://github.com/nodejs/node-gyp/pull/2497, - // and also see the issues/PRs linked from that one for more context. getNpmBuildFlags() { - return [`--target=${this.electronVersion}`, `--disturl=${config.getElectronUrl()}`, `--arch=${config.getElectronArch()}`, "--force-process-config"]; + // We used to set `--force-process-config=true` here, but we've upgraded + // Electron past the point where that's needed. For more details, see: + // https://github.com/nodejs/node-gyp/pull/2497. + return [`--target=${this.electronVersion}`, `--disturl=${config.getElectronUrl()}`, `--arch=${config.getElectronArch()}`]; } updateWindowsEnv(env) {