Closed
Bug 1391473
Opened 7 years ago
Closed 7 years ago
Add xz binaries to buildbot workers
Categories
(Release Engineering :: Release Automation: Other, enhancement, P1)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
References
Details
Attachments
(1 file, 2 obsolete files)
6.08 KB,
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
In devedition 56.0b4 build1 we hit errors like this: 14:09:04 INFO - MAR=../../dist/host/bin/mar.exe \ 14:09:04 INFO - MOZ_PRODUCT_VERSION=56.0 \ 14:09:04 INFO - c:/builds/moz2_slave/rel-m-beta_de_w32_l10n_rpk-000/build/mozilla-beta/tools/update-packaging/make_full_update.sh \ 14:09:04 INFO - '../../dist/update/firefox-56.0.ach.win32.complete.mar' \ 14:09:04 INFO - 'c:/builds/moz2_slave/rel-m-beta_de_w32_l10n_rpk-000/build/mozilla-beta/obj-l10n/dist/l10n-stage/firefox' 14:09:04 INFO - xz was not found on this system! 14:09:04 INFO - exiting The windows l10n jobs still happen on buildbot machines so we need to have xz available there for bug 1385780. I'm looking at adding it to tooltool and adjusting the PATH.
Assignee | ||
Comment 1•7 years ago
|
||
Uploaded https://tukaani.org/xz/xz-5.2.3-windows.zip to tooltool, sha512 1830819d78fb8bf164000fea0e79f2268509bc2557ba302eb1b30b73f306e6122ab478edd854ae688e51d9378e03ed57c56c0e7399bbb04509c46c6fc7718692.
Assignee | ||
Comment 2•7 years ago
|
||
Redid that to add a xz-5.2.3 prefix so that it unpacks nicely, sha512 8c34a608c83524d3ec3c9ef7f71b94ada9775bf2d7fdc843006e84205760157ae71bfa1f5836782716bbfaf63f2d003b1bb5e45f132bfbdaed3dfe5f10e8ae99.
Assignee | ||
Comment 3•7 years ago
|
||
This is untested but I think it should work. Will test while builds are running.
Attachment #8898553 -
Flags: review?(aki)
Assignee | ||
Updated•7 years ago
|
Attachment #8898553 -
Attachment is obsolete: true
Attachment #8898553 -
Flags: review?(aki)
Assignee | ||
Comment 4•7 years ago
|
||
This is the same approach Thunderbird used in bug 1387227 and avoids fighting your way through a thicket of functions to make sure the interpolation works.
Attachment #8898558 -
Flags: review?(aki)
Assignee | ||
Comment 5•7 years ago
|
||
Comment on attachment 8898558 [details] [diff] [review] [gecko] Add xz for windows l10n repacks, v2 This doesn't work in the release repack script.
Attachment #8898558 -
Attachment is obsolete: true
Attachment #8898558 -
Flags: review?(aki)
Updated•7 years ago
|
Attachment #8898594 -
Flags: review?(aki) → review+
Assignee | ||
Comment 7•7 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/80312a6daa088138cc3d53b3944cd814759a623f
Pushed by nthomas@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/575f17791f38 Add xz binaries to buildbot workers, r=aki
Assignee | ||
Comment 9•7 years ago
|
||
Green l10n jobs for deved 56.0b4 build2 on 80312a6daa08.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/575f17791f38
Comment 11•7 years ago
|
||
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/b6bb6c4cad18 Keep tooltool manifests in sync (Port bug 1391473: Add xz binaries to buildbot workers). rs=bustage-fix DONTBUILD
You need to log in
before you can comment on or make changes to this bug.
Description
•