Bug 1381577 - Part V; Remove target.complete.mar from OSX l10n signing (its signed as part of repackage) and error out if an unknown platform attempts to get signed from l10n. r=kmoir draft
authorJustin Wood <Callek@gmail.com>
Tue, 18 Jul 2017 14:29:46 -0400
changeset 613364 a9236c910f5c60b4c11d9c9ff7491b42b4d6b98b
parent 613363 9a8919eb2274e808aa61a6b5f0f0e8d8230cfd2b
child 613365 d86730b3fd30e3cbecef93cdda634466156d2f2e
push id69770
push userCallek@gmail.com
push dateFri, 21 Jul 2017 20:09:14 +0000
reviewerskmoir
bugs1381577
milestone56.0a1
Bug 1381577 - Part V; Remove target.complete.mar from OSX l10n signing (its signed as part of repackage) and error out if an unknown platform attempts to get signed from l10n. r=kmoir Land date changes to support windows nightlies onto central MozReview-Commit-ID: 9lWSEbQznyr
taskcluster/taskgraph/transforms/nightly_l10n_signing.py
--- a/taskcluster/taskgraph/transforms/nightly_l10n_signing.py
+++ b/taskcluster/taskgraph/transforms/nightly_l10n_signing.py
@@ -26,34 +26,34 @@ def make_signing_description(config, job
             job_specs = [
                 {
                     'artifacts': ['public/build/{locale}/target.apk'],
                     'format': 'jar',
                 },
             ]
         elif 'macosx' in dep_platform:
             job_specs = [
-                 {
+                {
                     'artifacts': ['public/build/{locale}/target.dmg'],
                     'format': 'macapp',
-                 }, {
-                    'artifacts': ['public/build/{locale}/target.complete.mar'],
-                    'format': 'mar',
-                 }
+                }
             ]
-        else:
+        elif 'linux' in dep_platform:
             job_specs = [
                 {
                     'artifacts': ['public/build/{locale}/target.tar.bz2'],
                     'format': 'gpg',
                 }, {
                     'artifacts': ['public/build/{locale}/target.complete.mar'],
                     'format': 'mar',
                 }
             ]
+        else:
+            raise Exception("Platform not implemented for signing")
+
         upstream_artifacts = []
         for spec in job_specs:
             fmt = spec['format']
             upstream_artifacts.append({
                 "taskId": {"task-reference": "<unsigned-repack>"},
                 "taskType": "l10n",
                 # Set paths based on artifacts in the specs (above) one per
                 # locale present in the chunk this is signing stuff for.