Merge pull request #1 from agilebits/bemini/#17-limit-lipo

Don’t lipo fat binaries or files that are the same arch.
electron · Apr 29, 2021 · a0d7f04 · a0d7f04
2 parents b445fa1 + 55ac932
commit a0d7f04
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -114,6 +114,17 @@ export const makeUniversalApp = async (opts: MakeUniversalOpts): Promise<void> =
       const first = await fs.realpath(path.resolve(tmpApp, machOFile.relativePath));
       const second = await fs.realpath(path.resolve(opts.arm64AppPath, machOFile.relativePath));
 
+      const x64Sha = await sha(path.resolve(opts.x64AppPath, machOFile.relativePath));
+      const arm64Sha = await sha(path.resolve(opts.arm64AppPath, machOFile.relativePath));
+      if (x64Sha === arm64Sha) {
+        d(
+          'SHA for Mach-O file',
+          machOFile.relativePath,
+          `matches across builds ${x64Sha}===${arm64Sha}, skipping lipo`,
+        );
+        continue;
+      }
+
       d('joining two MachO files with lipo', {
         first,
         second,