Closed Bug 1066123 Opened 5 years ago Closed 5 years ago
Fix b2g desktop osx due to v2 signature changes for Firefox
Although the file structure for Firefox.app has changed to support v2 signatures on OSX, the structure for b2g desktop OSX bundles hasn't changed (yet). We need to ensure that packager.mk continues to do the right thing for b2g desktop OSX builds.
Will send to try shortly to confirm.
Turns out that this will need significantly more work, and a package layout like the Firefox.app after all. Looking into it.
Summary: Ensure that packager.mk does the right thing for b2g desktop OSX after the v2 changes to Firefox OSX bundles → Fix b2g desktop osx due to v2 signature changes for Firefox.app
Attachment #8487966 - Attachment is obsolete: true
Comment on attachment 8488390 [details] [diff] [review] Patch Try turned green.
Comment on attachment 8488390 [details] [diff] [review] Patch >diff --git a/b2g/installer/package-manifest.in b/b2g/installer/package-manifest.in >--- a/b2g/installer/package-manifest.in >+++ b/b2g/installer/package-manifest.in >@@ -37,17 +37,16 @@ > #endif > @BINPATH@/dictionaries/* > @BINPATH@/hyphenation/* > #ifdef XP_WIN32 > @BINPATH@/uninstall/helper.exe > #endif > > [xpcom] >-@BINPATH@/dependentlibs.list removing this will break other builds
True, that should probably have been ifdef'd to OSX. Will look into it. Which builds failed?
Comment on attachment 8489019 [details] [diff] [review] Patch Try run is green (updater xpcshell test failures are handled in separate bugs): https://tbpl.mozilla.org/?tree=Try&rev=3b9892dd2636
Attachment #8489019 - Flags: review?(ted)
(In reply to Stephen Pohl [:spohl] from comment #6) > True, that should probably have been ifdef'd to OSX. Will look into it. > > Which builds failed? The build would succeed but running the build or tests would fail since that file is needed. B2G ICS Emulator Opt was failing tests for me. Thanks!
Attachment #8489019 - Flags: review?(ted) → review+
Pushed to fx-team https://hg.mozilla.org/integration/fx-team/rev/47c2f60b1174
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Landed on aurora in the Mac V2 signing combined patch in bug 1047584
You need to log in before you can comment on or make changes to this bug.