Closed Bug 1386402 Opened 7 years ago Closed 7 years ago

Change mar generation scripts to exit with an error if xz is not found on the system

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(firefox56 fixed, firefox57 fixed)

RESOLVED FIXED
Tracking Status
firefox56 --- fixed
firefox57 --- fixed

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file)

Over irc catlee asked for this.
Attached patch patchSplinter Review
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Attachment #8892630 - Flags: review?(bhearsum)
Attachment #8892630 - Flags: review?(bhearsum) → review+
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/46d7ac61d7cc
Change mar generation scripts to exit with an error if xz is not found on the system. r=bhearsum
Comment on attachment 8892630 [details] [diff] [review]
patch

This is for a build script so this request might not be necessary.

Approval Request Comment
[Feature/Bug causing the regression]: The check for the executable that compresses mar files never existed but now that we changed to using xz it is a good thing to have this check since our build systems are less likely to have xz installed.
[User impact if declined]: None
[Is this code covered by automated tests?]: None
[Has the fix been verified in Nightly?]: Will be used to generate updates today.
[Needs manual test from QE? If yes, steps to reproduce]: N/A
[List of other uplifts needed for the feature/fix]: N/A
[Is the change risky?]: No
[Why is the change risky/not risky?]: It is for a build script
[String changes made/needed]: None
Attachment #8892630 - Flags: approval-mozilla-beta?
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #3)
> Comment on attachment 8892630 [details] [diff] [review]
> patch
> 
> This is for a build script so this request might not be necessary.
> 
> Approval Request Comment
> [Feature/Bug causing the regression]: The check for the executable that
> compresses mar files never existed but now that we changed to using xz it is
> a good thing to have this check since our build systems are less likely to
> have xz installed.
> [User impact if declined]: None
> [Is this code covered by automated tests?]: None
> [Has the fix been verified in Nightly?]: Will be used to generate updates
> today.
Actually, this hasn't been merged to m-c yet and it will be used to generate updates after it has been merged.
https://hg.mozilla.org/mozilla-central/rev/46d7ac61d7cc
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Do we still need to uplift this for beta 1?
Flags: needinfo?(robert.strong.bugs)
Not for beta 1. This should land at the same time as the other server side mar generation scripts in Bug 1387241.
Flags: needinfo?(robert.strong.bugs)
Noting this (and the work in bug 1387241) is now waiting to land after we build beta 3. Not for beta 3.
Comment on attachment 8892630 [details] [diff] [review]
patch

Needed for 56 beta 3 to beta 4 watershed/migration to 64-bit.
Attachment #8892630 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.