treewide: don't accidentally package electron-dist into .asar file (#523841)

This commit is contained in:
dish
2026-05-26 17:51:57 +00:00
committed by GitHub
6 changed files with 27 additions and 45 deletions
+4 -3
View File
@@ -27,14 +27,15 @@ buildNpmPackage rec {
nativeBuildInputs = [ copyDesktopItems ];
postBuild = ''
cp -r ${electron.dist} electron-dist
chmod -R u+w electron-dist
electron_dist="$(mktemp -d)"
cp -r ${electron.dist}/. "$electron_dist"
chmod -R u+w "$electron_dist"
npm exec electron-builder -- \
--dir \
-c.npmRebuild=true \
-c.asarUnpack="**/*.node" \
-c.electronDist=electron-dist \
-c.electronDist="$electron_dist" \
-c.electronVersion=${electron.version}
'';
+4 -3
View File
@@ -34,12 +34,13 @@ buildNpmPackage (finalAttrs: {
};
postBuild = ''
cp -r ${electron.dist} electron-dist
chmod -R u+w electron-dist
electron_dist="$(mktemp -d)"
cp -r ${electron.dist}/. "$electron_dist"
chmod -R u+w "$electron_dist"
npm exec electron-builder -- \
--dir \
-c.electronDist=electron-dist \
-c.electronDist="$electron_dist" \
-c.electronVersion=${electron.version} \
--config electron-builder.config.js
'';
+5 -4
View File
@@ -86,8 +86,9 @@ stdenv.mkDerivation rec {
patchShebangs {node_modules,app/node_modules,backend/node_modules}
cp -r ${electron.dist} electron-dist
chmod -R u+w electron-dist
electron_dist="$(mktemp -d)"
cp -r ${electron.dist}/. "$electron_dist"
chmod -R u+w "$electron_dist"
runHook postConfigure
'';
@@ -98,7 +99,7 @@ stdenv.mkDerivation rec {
tsc && cd app && yarn --offline run build && cd ..
yarn --offline run electron-builder --dir \
-c.electronDist=electron-dist \
-c.electronDist="$electron_dist" \
-c.electronVersion=${electron.version}
runHook postBuild
@@ -143,7 +144,7 @@ stdenv.mkDerivation rec {
doCheck = true;
checkPhase = ''
export ELECTRON_OVERRIDE_DIST_PATH=electron-dist/
export ELECTRON_OVERRIDE_DIST_PATH="$electron_dist"
yarn test:app --offline
yarn test:backend --offline
+6 -14
View File
@@ -16,8 +16,6 @@
makeWrapper,
replaceVars,
vulkan-loader,
nixosTests,
}:
buildNpmPackage (finalAttrs: {
@@ -72,19 +70,13 @@ buildNpmPackage (finalAttrs: {
buildPhase = ''
runHook preBuild
# electronDist needs to be modifiable
cp -r ${electron.dist} electron-dist
chmod -R u+w electron-dist
''
# Electron builder complains about symlink in electron-dist
+ lib.optionalString stdenv.hostPlatform.isLinux ''
rm electron-dist/libvulkan.so.1
cp ${lib.getLib vulkan-loader}/lib/libvulkan.so.1 electron-dist
''
+ ''
electron_dist="$(mktemp -d)"
cp -r ${electron.dist}/. "$electron_dist"
chmod -R u+w "$electron_dist"
npm run build
npm exec electron-builder -- --dir \
-c.electronDist=electron-dist \
-c.electronDist="$electron_dist" \
-c.electronVersion=${electron.version}
runHook postBuild
@@ -123,7 +115,7 @@ buildNpmPackage (finalAttrs: {
doCheck = stdenv.hostPlatform.isLinux;
checkPhase = ''
runHook preCheck
export ELECTRON_OVERRIDE_DIST_PATH=electron-dist/
export ELECTRON_OVERRIDE_DIST_PATH="$electron_dist"
export PUPPETEER_EXECUTABLE_PATH=${chromium}/bin/chromium
export CHROME_BIN=${chromium}/bin/chromium
npm run test
+4 -10
View File
@@ -5,7 +5,6 @@
fetchFromGitHub,
makeWrapper,
electron_40,
vulkan-loader,
makeDesktopItem,
copyDesktopItems,
commandLineArgs ? [ ],
@@ -67,17 +66,12 @@ buildNpmPackage (finalAttrs: {
cp -r ${electron.dist} electron-dist
chmod -R u+w electron-dist
''
# Electron builder complains about symlink in electron-dist
+ lib.optionalString stdenv.hostPlatform.isLinux ''
rm electron-dist/libvulkan.so.1
cp '${lib.getLib vulkan-loader}/lib/libvulkan.so.1' electron-dist
''
# Explicitly set identity to null to avoid signing on arm64 macs with newer electron-builder.
# See: https://github.com/electron-userland/electron-builder/pull/9007
+ ''
npm run electron:pack
# Explicitly set identity to null to avoid signing on arm64 macs with newer electron-builder.
# See: https://github.com/electron-userland/electron-builder/pull/9007
./node_modules/.bin/electron-builder \
--dir \
--config .electron-builder.config.mjs \
+4 -11
View File
@@ -10,7 +10,6 @@
makeWrapper,
nix-update-script,
versionCheckHook,
vulkan-loader,
which,
}:
@@ -46,21 +45,15 @@ buildNpmPackage rec {
buildPhase = ''
runHook preBuild
cp -r ${electron_41.dist} electron-dist
chmod -R u+w electron-dist
''
# Electron builder complains about symlink in electron-dist
+ lib.optionalString stdenv.hostPlatform.isLinux ''
rm electron-dist/libvulkan.so.1
cp ${lib.getLib vulkan-loader}/lib/libvulkan.so.1 electron-dist
''
+ ''
electron_dist="$(mktemp -d)"
cp -r ${electron_41.dist}/. "$electron_dist"
chmod -R u+w "$electron_dist"
npm exec electron-builder -- \
--dir \
-c.npmRebuild=true \
-c.asarUnpack="**/*.node" \
-c.electronDist=electron-dist \
-c.electronDist="$electron_dist" \
-c.electronVersion=${electron_41.version} \
-c.mac.identity=null