Commit 8d1c4c39 authored by Sukhbir Singh's avatar Sukhbir Singh
Browse files

Bug 26319: Don't package up the whole Tor Browser in the `mach package` step

This commit sets `mach build stage-package` instead of `mach package` in
the Firefox build; this helps us to use the useful properties of `mach
package` but avoids creating the final DMG or EXE during the Firefox
build which we don't want as we do that later when building Tor Browser.

We no longer need to patch `installer.py` (bug 26205) but we still need
the patch for the uninstaller otherwise the Firefox build fails trying
to find the required NSIS files.

The no-dmg.patch for macOS is also not required and is removed (bug 24632).
parent 7bace2c6
......@@ -138,16 +138,12 @@ fi
patch -p1 < $rootdir/1467041.patch
[% END -%]
[% IF c("var/osx") %]
patch -p1 < $rootdir/no-dmg.patch
[% END -%]
rm -f configure
rm -f js/src/configure
./mach configure --with-tor-browser-version=[% c("var/torbrowser_version") %] --with-distribution-id=org.torproject --enable-update-channel=[% c("var/torbrowser_update_channel") %] --enable-bundled-fonts
./mach build
./mach package
./mach build stage-package
[% IF c("var/osx") %]
cp -a obj-macos/dist/firefox/* $distdir
......
......@@ -93,8 +93,6 @@ input_files:
enable: '[% c("var/selfrando") %]'
- filename: fix-info-plist.py
enable: '[% c("var/osx") %]'
- filename: no-dmg.patch
enable: '[% c("var/osx") %]'
- URL: https://people.torproject.org/~gk/mirrors/sources/msvcr100.dll
sha256sum: 8793353461826fbd48f25ea8b835be204b758ce7510db2af631b28850355bd18
enable: '[% c("var/windows-i686") %]'
......
From 7204fc9191a55dbbd3b94ee2d64ece8d9910cf23 Mon Sep 17 00:00:00 2001
From: Georg Koppen <gk@torproject.org>
Date: Thu, 31 May 2018 19:15:18 +0000
Subject: [PATCH] Don't create a .dmg yet
diff --git a/toolkit/mozapps/installer/upload-files.mk b/toolkit/mozapps/installer/upload-files.mk
index 8ace9703a160..82df89d1f431 100644
--- a/toolkit/mozapps/installer/upload-files.mk
+++ b/toolkit/mozapps/installer/upload-files.mk
@@ -216,14 +216,7 @@ ifeq ($(MOZ_PKG_FORMAT),DMG)
_ABS_MOZSRCDIR = $(shell cd $(MOZILLA_DIR) && pwd)
PKG_DMG_SOURCE = $(MOZ_PKG_DIR)
- INNER_MAKE_PACKAGE = \
- $(call py_action,make_dmg, \
- $(if $(MOZ_PKG_MAC_DSSTORE),--dsstore '$(MOZ_PKG_MAC_DSSTORE)') \
- $(if $(MOZ_PKG_MAC_BACKGROUND),--background '$(MOZ_PKG_MAC_BACKGROUND)') \
- $(if $(MOZ_PKG_MAC_ICON),--icon '$(MOZ_PKG_MAC_ICON)') \
- --volume-name '$(MOZ_APP_DISPLAYNAME)' \
- '$(PKG_DMG_SOURCE)' '$(PACKAGE)' \
- )
+ INNER_MAKE_PACKAGE = true
INNER_UNMAKE_PACKAGE = \
$(call py_action,unpack_dmg, \
$(if $(MOZ_PKG_MAC_DSSTORE),--dsstore '$(MOZ_PKG_MAC_DSSTORE)') \
--
2.17.0
From 4ef264b601ac92a0b385c40bd011ef94618a176a Mon Sep 17 00:00:00 2001
From e338a77c64c0afb72e857c7daa63346d35ad3891 Mon Sep 17 00:00:00 2001
From: Sukhbir Singh <sukhbir@torproject.org>
Date: Tue, 12 Jun 2018 21:58:24 -0400
Subject: [PATCH 1/1] Bug 26205: Don't build the uninstaller for Windows during
Firefox compilation
---
browser/Makefile.in | 4 ----
browser/installer/Makefile.in | 4 ----
.../mozbuild/mozbuild/repackaging/installer.py | 17 +----------------
3 files changed, 1 insertion(+), 24 deletions(-)
browser/Makefile.in | 4 ----
browser/installer/Makefile.in | 4 ----
2 files changed, 8 deletions(-)
diff --git a/browser/Makefile.in b/browser/Makefile.in
index 2eb9e708f68e..0c9442183942 100644
......@@ -40,34 +39,6 @@ index 1bf11252ee06..dee6d5434106 100644
ifeq (cocoa,$(MOZ_WIDGET_TOOLKIT))
MOZ_PKG_MAC_DSSTORE=$(topsrcdir)/$(MOZ_BRANDING_DIRECTORY)/dsstore
MOZ_PKG_MAC_BACKGROUND=$(topsrcdir)/$(MOZ_BRANDING_DIRECTORY)/background.png
diff --git a/python/mozbuild/mozbuild/repackaging/installer.py b/python/mozbuild/mozbuild/repackaging/installer.py
index 9b49eb555153..1e6f6e73c3d8 100644
--- a/python/mozbuild/mozbuild/repackaging/installer.py
+++ b/python/mozbuild/mozbuild/repackaging/installer.py
@@ -27,22 +27,7 @@ def repackage_installer(topsrcdir, tag, setupexe, package, output, package_name,
tmpdir = tempfile.mkdtemp()
old_cwd = os.getcwd()
try:
- if package:
- z = zipfile.ZipFile(package)
- z.extractall(tmpdir)
- z.close()
-
- # Copy setup.exe into the root of the install dir, alongside the
- # package.
- shutil.copyfile(setupexe, mozpath.join(tmpdir, mozpath.basename(setupexe)))
-
- # archive_exe requires us to be in the directory where the package is
- # unpacked (the tmpdir)
- os.chdir(tmpdir)
-
- sfx_package = mozpath.join(topsrcdir, sfx_stub)
-
- archive_exe(package_name, tag, sfx_package, output)
+ pass
finally:
os.chdir(old_cwd)
--
2.17.1
2.18.0.rc2
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment