1443144 - Snap: build fails missing mime*

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, defect)

Description

[Ken VanDine]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Firefox/59.0
Build ID: 20180226180053

Steps to reproduce:

Recent changes to the snapcraft.yaml requires some new files be copied into ${WORKSPACE}.  This causes a build failure missing mimeapps.list and mime-handler.xml

Comment 1

[Ken VanDine]

Attachment: Bug 1443144 - Fix snap build failure caused by files not being copied into ${WORKSPACE}

Review commit: https://reviewboard.mozilla.org/r/225004/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/225004/

Comment 2

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8956066 [details]
Bug 1443144 - Fix snap build failure caused by files not being copied into ${WORKSPACE}

https://reviewboard.mozilla.org/r/225004/#review230964

Sorry for missing that, in my previous reviews.

Comment 3

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8956066 [details]
Bug 1443144 - Fix snap build failure caused by files not being copied into ${WORKSPACE}

Approval Request Comment
[Feature/Bug causing the regression]: None
[User impact if declined]: Unable to ship the first Snap release. Snaps are a new package format powered by Canonical.
[Is this code covered by automated tests?]: N/A
[Has the fix been verified in Nightly?]: N/A. Releng crafts Snap on beta. Starting Firefox 59, Snap are also built on release.
[Needs manual test from QE? If yes, steps to reproduce]: No. This has been tested by Canonical's QA.
[List of other uplifts needed for the feature/fix]: N/A
[Is the change risky?]: No
[Why is the change risky/not risky?]: This is a Snap only fix for 59. It has no impact on the regular product.
[String changes made/needed]: None.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8956066 [details]
Bug 1443144 - Fix snap build failure caused by files not being copied into ${WORKSPACE}

Fix needed for Snap builds, which we want to ship for Fx59. Approved for 59rc1.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/058cc826edb6
Fix snap build failure caused by files not being copied into ${WORKSPACE} r=jlorenzo

Comment 6

[Johan Lorenzo [:jlorenzo]]

Thank you very much for the super quick approval, Ryan! 

I landed the patch on mozilla-release before it reached central: https://hg.mozilla.org/releases/mozilla-release/rev/5d05e188fe7ceb8d89d4aebdbc006ae79d0375d9. The reasons are:
* This code isn't run on mozilla-central, but only on beta and release
* Only linters run on push. I manually ran shellcheck to ensure nothing was broken.
* The code actually runs on mozilla-beta and release when builds are promoted.

Comment 7

[Johan Lorenzo [:jlorenzo]]

Attachment: Bug 1443144 - part2: Fix bad shell quotes a=RyanVM

Review commit: https://reviewboard.mozilla.org/r/225030/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/225030/

Comment 8

[Simon Fraser [:sfraser] ⌚️GMT]

Comment on attachment 8956092 [details]
Bug 1443144 - part2: Fix bad shell quotes  a=RyanVM

https://reviewboard.mozilla.org/r/225030/#review230980

ooh, nasty shell expansion, good catch

Comment 9

[Johan Lorenzo [:jlorenzo]]

(In reply to Simon Fraser [:sfraser] ⌚️GMT from comment #8)
> ooh, nasty shell expansion, good catch

Kudos to Ken who found it before GTB. I landed the fixup on m-r at https://hg.mozilla.org/releases/mozilla-release/rev/7f9be73fde97dd6831b2954bd37afe477ecb0573

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f00eccec1c83
part2: Fix bad shell quotes r=sfraser a=RyanVM

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/058cc826edb6

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

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