Bug 1562412 - address beetmover-geckoview hardcoded assumptions. r=jlorenzo

We should move beetmover-geckoview to the multi_dep loader. Until then, this should un-break us when we adjust build-signing deps. Unknown deps will still lead to declarative artifacts bustage.

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

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Aki Sasaki 2020-02-25 15:35:33 +00:00
Родитель 614ce54aa9
Коммит 74536ae2b6
2 изменённых файлов: 1 добавлений и 9 удалений

Просмотреть файл

@ -8,6 +8,7 @@ default_locales: # Ignored for geckoview
- en-US - en-US
tasktype_map: # Map task reference to task type. tasktype_map: # Map task reference to task type.
build: build build: build
build-fat-aar: build
build-signing: signing build-signing: signing
# A default entry, which the mappings below extend and override. # A default entry, which the mappings below extend and override.

Просмотреть файл

@ -112,15 +112,6 @@ def make_task_description(config, jobs):
@transforms.add @transforms.add
def make_task_worker(config, jobs): def make_task_worker(config, jobs):
for job in jobs: for job in jobs:
valid_beetmover_job = set(job['dependencies'].keys()) == {'build', 'build-signing'}
if not valid_beetmover_job:
raise NotImplementedError(
'Beetmover-geckoview must have 2 dependencies: build and build-signing. '
'Got: {}'.format(
job['dependencies']
)
)
job['worker'] = { job['worker'] = {
'artifact-map': generate_beetmover_artifact_map( 'artifact-map': generate_beetmover_artifact_map(
config, config,