1206379 - Nexus 5 L blobfree distribution is not picked up

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[:gerard-majax]

> testing/taskcluster/scripts/phone-builder/post-build.sh:if [ -f $WORKSPACE/B2G/upload-public/${DEVICE}.blobfree-dist.zip ]; then

We expecte hammerhead.blobfree-dist.zip but I don't know why in this script, we have DEVICE with |nexus| value.

Hence the execution gives:
> if [ -f $WORKSPACE/B2G/upload-public/${DEVICE}.blobfree-dist.zip ]; then
>   mv $WORKSPACE/B2G/upload-public/${DEVICE}.blobfree-dist.zip $HOME/artifacts-public
> fi
> ++ '[' -f /home/worker/workspace/B2G/upload-public/nexus.blobfree-dist.zip ']'

Comment 1

[:gerard-majax]

Removed ${DEVICE} in favor of * in bug 1206368 attachment 8663279 [details] [diff] [review]

Comment 2

[:gerard-majax]

Attachment: Fix blobfree distribution collect for nexus-5-l

Comment 3

[Wander Lairson Costa]

This is because the device name for flame-kk is flame, this patch will break flame-kk builds, iiuc.

Comment 4

[:gerard-majax]

(In reply to Wander Lairson Costa [:wcosta] from comment #3)
> This is because the device name for flame-kk is flame, this patch will break
> flame-kk builds, iiuc.

No, it does not break anything: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4f7de9c69ef

Comment 5

[Wander Lairson Costa]

Comment on attachment 8663367 [details] [diff] [review]
Fix blobfree distribution collect for nexus-5-l

Review of attachment 8663367 [details] [diff] [review]:
-----------------------------------------------------------------

You are right, this only affects blobfree.

Comment 6

[:gerard-majax]

Attachment: Fix blobfree distribution collect for nexus-5-l

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/becc50c3d9c9

Comment 8

[:gerard-majax]

Works well, except that VARIANT=user build do not boots :)

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/becc50c3d9c9