Closed
Bug 1200861
Opened 9 years ago
Closed 9 years ago
Lightning repacks get fooled by interfaces.manifest generated from make export
Categories
(Calendar :: Build Config, defect)
Calendar
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
4.3
People
(Reporter: Fallen, Assigned: Fallen)
References
Details
Attachments
(1 file)
3.20 KB,
patch
|
mkmelin
:
review+
Fallen
:
approval-calendar-aurora+
Fallen
:
approval-calendar-beta+
|
Details | Diff | Splinter Review |
When doing l10n repacks on beta, make export is now called, which generates objdir-l10n/dist/xpi-stage/lightning/components/interfaces.manifest. The current lightning packager checks for the existence of objdir-l10n/dist/xpi-stage/lightning so it will work when not called from the l10n repacks but locally, without overwriting the generated lightning. The upcoming patch will check if EN_US_BINARY_URL is finally set and then go about downloading and unpacking unconditionally. I've tested this locally with a script that emulates the l10n repacks, for the case that xpi-stage/lightning does not exist at all, and for the case where it contains just the interfaces.manifest.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8655710 -
Flags: review?(mkmelin+mozilla)
Attachment #8655710 -
Flags: approval-calendar-beta+
Attachment #8655710 -
Flags: approval-calendar-aurora+
Comment 2•9 years ago
|
||
Comment on attachment 8655710 [details] [diff] [review] Fix - v1 Review of attachment 8655710 [details] [diff] [review]: ----------------------------------------------------------------- Looks reasonable to me
Attachment #8655710 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Pushed to comm-central changeset a69ff7e5e57b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.5
Assignee | ||
Comment 4•9 years ago
|
||
Backported to releases/comm-aurora changeset 39de7d8e88d4
Target Milestone: 4.5 → 4.4
Assignee | ||
Comment 5•9 years ago
|
||
Backported to releases/comm-beta changeset 88062c7a9dc0
Target Milestone: 4.4 → 4.3
Comment 6•9 years ago
|
||
All builds are broken on all 3 branches: /builds/slave/tb-c-cen-lx-000000000000000000/build/calendar/lightning/lightning-packager.mk:58: *** You must set EN_US_BINARY_URL. Stop. make[4]: *** [calendar/providers/gdata/export] Error 2
Flags: needinfo?(philipp)
Assignee | ||
Comment 7•9 years ago
|
||
Sorry about that. Taking care!
Status: RESOLVED → REOPENED
Flags: needinfo?(philipp)
Resolution: FIXED → ---
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 16•9 years ago
|
||
I've pushed a bustage fix with rs=Callek: https://hg.mozilla.org/comm-central/rev/6bcf01ef7591 Waiting for green c-c before I backport this.
Assignee | ||
Comment 17•9 years ago
|
||
Linux is green, that is good enough for me. Here we go: https://hg.mozilla.org/releases/comm-aurora/rev/069a4fdedeb1 https://hg.mozilla.org/releases/comm-beta/rev/409957bf2393
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•