From fe42d8e5715eb1c2bfffe72693cdb358c7b9ab1d Mon Sep 17 00:00:00 2001 From: develar Date: Fri, 23 Jun 2017 20:05:37 +0200 Subject: [PATCH] feat: ignore dll/exe files from node_modules if target platform not windows Close #1738 --- .idea/dictionaries/develar.xml | 2 ++ docs/Options.md | 2 +- package.json | 4 +-- packages/electron-builder/src/fileMatcher.ts | 36 ++++++++++++++----- .../electron-builder/src/platformPackager.ts | 14 ++++---- test/out/__snapshots__/BuildTest.js.snap | 23 ++++++++---- test/src/BuildTest.ts | 36 +++++++++++++------ test/src/helpers/packTester.ts | 3 +- yarn.lock | 16 ++++++--- 9 files changed, 94 insertions(+), 42 deletions(-) diff --git a/.idea/dictionaries/develar.xml b/.idea/dictionaries/develar.xml index b4720461bf2..f81b400a75d 100644 --- a/.idea/dictionaries/develar.xml +++ b/.idea/dictionaries/develar.xml @@ -44,6 +44,7 @@ cpus createconfig crypto + csproj ctype cuint customfunction @@ -257,6 +258,7 @@ xamarin xenial xorriso + xproj xubuntu yargs zenity diff --git a/docs/Options.md b/docs/Options.md index 447aeda472f..dd7749a8e74 100644 --- a/docs/Options.md +++ b/docs/Options.md @@ -76,7 +76,7 @@ You can use macros in the file patterns, artifact file name patterns and publish * `**/*` * `!**/node_modules/*/{CHANGELOG.md,README.md,README,readme.md,readme,test,__tests__,tests,powered-test,example,examples,*.d.ts}` * `!**/node_modules/.bin` -* `!**/*.{o,hprof,orig,pyc,pyo,rbc}` +* `!**/*.{iml,o,hprof,orig,pyc,pyo,rbc,swp,csproj,sln,xproj}` * `!**/._*` * `!**/{.DS_Store,.git,.hg,.svn,CVS,RCS,SCCS,__pycache__,thumbs.db,.gitignore,.gitattributes,.editorconfig,.flowconfig,.yarn-metadata.json,.idea,.vs,appveyor.yml,.travis.yml,circle.yml,npm-debug.log,.nyc_output,yarn.lock,.yarn-integrity}` diff --git a/package.json b/package.json index af6e6235c3d..1c30cd6ac8d 100644 --- a/package.json +++ b/package.json @@ -66,10 +66,10 @@ }, "devDependencies": { "@types/ini": "^1.3.29", - "@types/jest": "^20.0.1", + "@types/jest": "^20.0.2", "@types/js-yaml": "^3.5.31", "@types/source-map-support": "^0.4.0", - "@types/xml2js": "^0.0.33", + "@types/xml2js": "^0.4.0", "babel-plugin-array-includes": "^2.0.3", "babel-plugin-transform-async-to-module-method": "^6.24.1", "babel-plugin-transform-es2015-destructuring": "^6.23.0", diff --git a/packages/electron-builder/src/fileMatcher.ts b/packages/electron-builder/src/fileMatcher.ts index 5f35e5c345a..1f0ad09b316 100644 --- a/packages/electron-builder/src/fileMatcher.ts +++ b/packages/electron-builder/src/fileMatcher.ts @@ -4,8 +4,9 @@ import { copyDir, copyOrLinkFile, Filter, statOrNull } from "electron-builder-ut import { mkdirs } from "fs-extra-p" import { Minimatch } from "minimatch" import * as path from "path" +import { Platform } from "./core" import { Config, FilePattern, PlatformSpecificBuildOptions } from "./metadata" -import { BuildInfo } from "./packagerApi" +import { PlatformPackager } from "./platformPackager" import { createFilter, hasMagic } from "./util/filter" /** @internal */ @@ -13,7 +14,7 @@ export class FileMatcher { readonly from: string readonly to: string - private readonly patterns: Array + readonly patterns: Array constructor(from: string, to: string, private readonly macroExpander: (pattern: string) => string, patterns?: Array | string | n) { this.from = macroExpander(from) @@ -86,31 +87,42 @@ export class FileMatcher { } /** @internal */ -export function createFileMatcher(info: BuildInfo, appDir: string, resourcesPath: string, macroExpander: (pattern: string) => string, platformSpecificBuildOptions: PlatformSpecificBuildOptions, buildResourceDir: string) { - const patterns = info.isPrepackedAppAsar ? null : getFileMatchers(info.config, "files", appDir, path.join(resourcesPath, "app"), false, macroExpander, platformSpecificBuildOptions) +export function createFileMatcher(appDir: string, resourcesPath: string, macroExpander: (pattern: string) => string, platformSpecificBuildOptions: PlatformSpecificBuildOptions, packager: PlatformPackager) { + const buildResourceDir = path.resolve(packager.info.projectDir, packager.buildResourcesDir) + + const patterns = packager.info.isPrepackedAppAsar ? null : getFileMatchers(packager.info.config, "files", appDir, path.join(resourcesPath, "app"), false, macroExpander, platformSpecificBuildOptions) const matcher = patterns == null ? new FileMatcher(appDir, path.join(resourcesPath, "app"), macroExpander) : patterns[0] const relativeBuildResourceDir = path.relative(matcher.from, buildResourceDir) const ignoreBuildResourceDirPattern = (relativeBuildResourceDir.length !== 0 && !relativeBuildResourceDir.startsWith(".")) ? `!${relativeBuildResourceDir}{,/**/*}` : null + const customFirstPatterns: Array = [] if (matcher.isEmpty() || matcher.containsOnlyIgnore()) { if (ignoreBuildResourceDirPattern != null) { matcher.addPattern(ignoreBuildResourceDirPattern) } - matcher.prependPattern("**/*") + customFirstPatterns.push("**/*") } else { if (ignoreBuildResourceDirPattern != null) { - matcher.prependPattern(ignoreBuildResourceDirPattern) + customFirstPatterns.push(ignoreBuildResourceDirPattern) } + // prependPattern - user pattern should be after to be able to override - matcher.prependPattern("**/node_modules/**/*") + customFirstPatterns.push("**/node_modules/**/*") matcher.addPattern("package.json") } + + if (packager.platform !== Platform.WINDOWS) { + // https://github.com/electron-userland/electron-builder/issues/1738 + customFirstPatterns.push("!**/node_modules/**/*.{dll,exe}") + } + + matcher.patterns.unshift(...customFirstPatterns) + matcher.addPattern("!**/node_modules/*/{CHANGELOG.md,ChangeLog,changelog.md,README.md,README,readme.md,readme,test,__tests__,tests,powered-test,example,examples,*.d.ts}") matcher.addPattern("!**/node_modules/.bin") - matcher.addPattern("!**/*.{o,hprof,orig,pyc,pyo,rbc,swp}") + matcher.addPattern(`!**/*.{iml,o,hprof,orig,pyc,pyo,rbc,swp,csproj,sln,xproj}`) matcher.addPattern("!**/._*") - matcher.addPattern("!*.iml") //noinspection SpellCheckingInspection matcher.addPattern("!**/{.git,.hg,.svn,CVS,RCS,SCCS," + "__pycache__,.DS_Store,thumbs.db,.gitignore,.gitattributes," + @@ -120,6 +132,12 @@ export function createFileMatcher(info: BuildInfo, appDir: string, resourcesPath "appveyor.yml,.travis.yml,circle.yml," + ".nyc_output}") + // https://github.com/electron-userland/electron-builder/issues/1738#issuecomment-310729208 + matcher.addPattern("!**/node_modules/lzma-native/build/**/*") + matcher.addPattern("**/node_modules/lzma-native/build/{Release,Debug}") + matcher.addPattern("!**/node_modules/lzma-native/deps/xz-*") + matcher.addPattern("!**/node_modules/lzma-native/deps/doc{,/**/*}") + return matcher } diff --git a/packages/electron-builder/src/platformPackager.ts b/packages/electron-builder/src/platformPackager.ts index 121ff5b39e2..a2d2b65ede0 100644 --- a/packages/electron-builder/src/platformPackager.ts +++ b/packages/electron-builder/src/platformPackager.ts @@ -165,7 +165,13 @@ export abstract class PlatformPackager } } - const defaultMatcher = createFileMatcher(this.info, appDir, resourcesPath, macroExpander, platformSpecificBuildOptions, path.resolve(this.info.projectDir, this.buildResourcesDir)) + const packContext: AfterPackContext = { + appOutDir, outDir, arch, targets, + packager: this, + electronPlatformName: platformName, + } + + const defaultMatcher = createFileMatcher(appDir, resourcesPath, macroExpander, platformSpecificBuildOptions, this) const isElectronCompile = asarOptions != null && isElectronCompileUsed(this.info) if (isElectronCompile) { defaultMatcher.addPattern("!.cache{,/**/*}") @@ -190,12 +196,6 @@ export abstract class PlatformPackager promise = new AsarPackager(appDir, resourcesPath, asarOptions, fileMatcher == null ? null : fileMatcher.createFilter(), transformer).pack(filter, isElectronCompile, this) } - const packContext: AfterPackContext = { - appOutDir, outDir, arch, targets, - packager: this, - electronPlatformName: platformName, - } - //noinspection ES6MissingAwait const promises = [promise, unlinkIfExists(path.join(resourcesPath, "default_app.asar")), unlinkIfExists(path.join(appOutDir, "version")), this.postInitApp(packContext)] if (this.platform !== Platform.MAC) { diff --git a/test/out/__snapshots__/BuildTest.js.snap b/test/out/__snapshots__/BuildTest.js.snap index 0a244361eea..9875ac1ec37 100644 --- a/test/out/__snapshots__/BuildTest.js.snap +++ b/test/out/__snapshots__/BuildTest.js.snap @@ -270,19 +270,32 @@ Object { } `; -exports[`relative index 1`] = ` +exports[`posix smart unpack 1`] = ` Object { "linux": Array [], } `; -exports[`smart unpack 1`] = ` +exports[`posix smart unpack 2`] = ` +Array [ + "app.asar", + "electron.asar", +] +`; + +exports[`relative index 1`] = ` Object { "linux": Array [], } `; -exports[`smart unpack 2`] = ` +exports[`win smart unpack 1`] = ` +Object { + "win": Array [], +} +`; + +exports[`win smart unpack 2`] = ` Array [ "app.asar", "electron.asar", @@ -296,14 +309,10 @@ Array [ "app.asar.unpacked/node_modules/edge-cs/src", "app.asar.unpacked/node_modules/edge-cs/src/edge-cs", "app.asar.unpacked/node_modules/edge-cs/src/edge-cs/EdgeCompiler.cs", - "app.asar.unpacked/node_modules/edge-cs/src/edge-cs/edge-cs.csproj", - "app.asar.unpacked/node_modules/edge-cs/src/edge-cs/edge-cs.sln", "app.asar.unpacked/node_modules/edge-cs/src/edge-cs/Properties", "app.asar.unpacked/node_modules/edge-cs/src/edge-cs/Properties/AssemblyInfo.cs", "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp", "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/EdgeCompiler.cs", - "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/edge-cs-coreclr.sln", - "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/edge-cs-coreclr.xproj", "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/gulpfile.js", "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/package.json", "app.asar.unpacked/node_modules/edge-cs/src/Edge.js.CSharp/project.json", diff --git a/test/src/BuildTest.ts b/test/src/BuildTest.ts index 93b6c5b8e92..e11249b8749 100644 --- a/test/src/BuildTest.ts +++ b/test/src/BuildTest.ts @@ -170,8 +170,9 @@ test.ifLinuxOrDevMac("beforeBuild", () => { }) }) -test.ifDevOrLinuxCi("smart unpack", app({ - targets: linuxDirTarget, +// https://github.com/electron-userland/electron-builder/issues/1738 +test.ifAll.ifDevOrLinuxCi("win smart unpack", app({ + targets: Platform.WINDOWS.createTarget(DIR_TARGET), }, { installDepsBefore: true, projectDirCreated: packageJson(it => { @@ -182,16 +183,31 @@ test.ifDevOrLinuxCi("smart unpack", app({ "@electron-builder/test-smart-unpack-empty": "1.0.0", } }), - packed: async context => { - const resourceDir = context.getResources(Platform.LINUX) - expect(await readAsarJson(path.join(resourceDir, "app.asar"), "node_modules/debug/package.json")).toMatchObject({ - name: "debug" - }) - - expect((await walk(resourceDir, file => !path.basename(file).startsWith("."))).map(it => it.substring(resourceDir.length + 1))).toMatchSnapshot() - } + packed: context => verifySmartUnpack(context.getResources(Platform.WINDOWS)) })) +async function verifySmartUnpack(resourceDir: string) { + expect(await readAsarJson(path.join(resourceDir, "app.asar"), "node_modules/debug/package.json")).toMatchObject({ + name: "debug" + }) + + expect((await walk(resourceDir, file => !path.basename(file).startsWith("."))).map(it => it.substring(resourceDir.length + 1))).toMatchSnapshot() +} + +// https://github.com/electron-userland/electron-builder/issues/1738 +test.ifAll.ifDevOrLinuxCi("posix smart unpack", app({ + targets: linuxDirTarget, +}, { + installDepsBefore: true, + projectDirCreated: packageJson(it => { + it.dependencies = { + "debug": "^2.2.0", + "edge-cs": "1.2.1", + "lzma-native": "2.0.3", + } + }), + packed: context => verifySmartUnpack(context.getResources(Platform.LINUX))})) + test("wine version", async () => { await checkWineVersion(BluebirdPromise.resolve("1.9.23 (Staging)")) await checkWineVersion(BluebirdPromise.resolve("2.0-rc2")) diff --git a/test/src/helpers/packTester.ts b/test/src/helpers/packTester.ts index 4e478fab260..aeb8f1b3dac 100644 --- a/test/src/helpers/packTester.ts +++ b/test/src/helpers/packTester.ts @@ -97,7 +97,8 @@ export async function assertPack(fixtureName: string, packagerOptions: PackagerO if (projectDirCreated != null) { await projectDirCreated(projectDir) if (checkOptions.installDepsBefore) { - await spawn("node", [path.join(__dirname, "..", "..", "vendor", "yarn.js"), "install", "--production", "--no-bin-links"], { + // bin links required (e.g. for node-pre-gyp - if package refers to it in the install script) + await spawn("node", [path.join(__dirname, "..", "..", "vendor", "yarn.js"), "install", "--production", "--no-lockfile"], { cwd: projectDir, }) } diff --git a/yarn.lock b/yarn.lock index df852eb254d..2a63f41500a 100644 --- a/yarn.lock +++ b/yarn.lock @@ -26,7 +26,7 @@ version "1.3.29" resolved "https://registry.yarnpkg.com/@types/ini/-/ini-1.3.29.tgz#1325e981e047d40d13ce0359b821475b97741d2f" -"@types/jest@^20.0.1": +"@types/jest@^20.0.2": version "20.0.2" resolved "https://registry.yarnpkg.com/@types/jest/-/jest-20.0.2.tgz#86c751121fb53dbd39bb1a08c45083da13f2dc67" @@ -44,9 +44,11 @@ dependencies: "@types/node" "*" -"@types/xml2js@^0.0.33": - version "0.0.33" - resolved "https://registry.yarnpkg.com/@types/xml2js/-/xml2js-0.0.33.tgz#20c5dd6460245284d64a55690015b95e409fb7de" +"@types/xml2js@^0.4.0": + version "0.4.0" + resolved "https://registry.yarnpkg.com/@types/xml2js/-/xml2js-0.4.0.tgz#e54a89a0055d5ed69305b2610f970909bf363e45" + dependencies: + "@types/node" "*" abab@^1.0.3: version "1.0.3" @@ -526,10 +528,14 @@ balanced-match@^1.0.0: version "1.0.0" resolved "https://registry.yarnpkg.com/balanced-match/-/balanced-match-1.0.0.tgz#89b4d199ab2bee49de164ea02b89ce462d71b767" -base64-js@1.2.0, base64-js@^1.0.2: +base64-js@1.2.0: version "1.2.0" resolved "https://registry.yarnpkg.com/base64-js/-/base64-js-1.2.0.tgz#a39992d723584811982be5e290bb6a53d86700f1" +base64-js@^1.0.2: + version "1.2.1" + resolved "https://registry.yarnpkg.com/base64-js/-/base64-js-1.2.1.tgz#a91947da1f4a516ea38e5b4ec0ec3773675e0886" + bcrypt-pbkdf@^1.0.0: version "1.0.1" resolved "https://registry.yarnpkg.com/bcrypt-pbkdf/-/bcrypt-pbkdf-1.0.1.tgz#63bc5dcb61331b92bc05fd528953c33462a06f8d"