tor-browser

commit d39aabe0d530c14272a2038e85b946189136a577
parent c90eaeb7af65a9a015b60c211c30665a73197fad
Author: Stephen King <sking@thunderbird.net>
Date:   Tue, 28 Oct 2025 18:22:52 +0000

Bug 1991990: Update MSIX Packaging Branding r=bhearsum

Thunderbird already uses a " Beta" suffix in its branding. Only add a " Beta" suffix if one does not already exist.

Also adds an esr channel with a corresponding " ESR" suffix if one does not aleady exist in the products branding.

Differential Revision: https://phabricator.services.mozilla.com/D268806

Diffstat:
M
python/mozbuild/mozbuild/mach_commands.py
 | 
2
+-
M
python/mozbuild/mozbuild/repackaging/msix.py
 | 
25
+++++++++++++++++++++----

2 files changed, 22 insertions(+), 5 deletions(-)
diff --git a/python/mozbuild/mozbuild/mach_commands.py b/python/mozbuild/mozbuild/mach_commands.py
@@ -2988,7 +2988,7 @@ def repackage_msi(
 @CommandArgument(
     "--channel",
     type=str,
-    choices=["official", "beta", "aurora", "nightly", "unofficial"],
+    choices=["official", "beta", "esr", "aurora", "nightly", "unofficial"],
     help="Release channel.",
 )
 @CommandArgument(
diff --git a/python/mozbuild/mozbuild/repackaging/msix.py b/python/mozbuild/mozbuild/repackaging/msix.py
@@ -354,6 +354,7 @@ def repackage_msix(
     if channel not in (
         "official",
         "beta",
+        "esr",
         "aurora",
         "nightly",
         "unofficial",
@@ -412,9 +413,17 @@ def repackage_msix(
     if not displayname:
         displayname = f"Mozilla {first}"
 
+        # Release (official) and Beta share branding.  Differentiate Beta a little bit.
         if channel == "beta":
-            # Release (official) and Beta share branding.  Differentiate Beta a little bit.
-            displayname += " Beta"
+            suffix = " Beta"
+            if not displayname.endswith(suffix):
+                displayname += suffix
+
+        elif channel == "esr":
+            # Release (official) and ESR share branding.  Differentiate ESR a little bit.
+            suffix = " ESR"
+            if not displayname.endswith(suffix):
+                displayname += suffix
 
     second = next(values)
     vendor = vendor or second
@@ -472,9 +481,17 @@ def repackage_msix(
     _, _, brandFullName = brandFullName.partition("=")
     brandFullName = brandFullName.strip()
 
+    # Release (official) and Beta share branding.  Differentiate Beta a little bit.
     if channel == "beta":
-        # Release (official) and Beta share branding.  Differentiate Beta a little bit.
-        brandFullName += " Beta"
+        suffix = " Beta"
+        if not brandFullName.endswith(suffix):
+            brandFullName += suffix
+
+    elif channel == "esr":
+        # Release (official) and ESR share branding.  Differentiate ESR a little bit.
+        suffix = " ESR"
+        if not brandFullName.endswith(suffix):
+            brandFullName += suffix
 
     branding = get_branding(
         use_official_branding, topsrcdir, build_app, unpack_finder, log