Closed Bug 1415057 Opened 3 years ago Closed 3 years ago

Linux 32 L10N repack failures on beta release builds


(Thunderbird :: Build Config, enhancement)

Not set


(Not tracked)



(Reporter: tomprince, Assigned: tomprince)




(2 files)

We disabled configuring stylo in for l10n-repacks. But now that stylo is enabled by default, that way disabling stylo no longer works.

A better way of handling things might be to add back --disable-compile-environment which was disabled in Bug 1311045
Before making changes to disable-compile-environment, please read bug 1345422 comment #57 and bug 1345422 comment #58. There appears to be some evidence that with disable-compile-environment in place, things start failing on Linux. Maybe it would be different in today's environment.

There has been a long history of adding and removing that option on the various platforms.
firefox has add `build/` that handles passing that argument, and unsetting a bunch of variables that will cause errors like the ones that have been seen with --disable-compile-environment in the past. (Added in Bug 1373150)
Pushed by
Disable compiler environment on L10N repacks; r=me
Closed: 3 years ago
Resolution: --- → FIXED
Attached patch Patch for 57.0b1Splinter Review
In an excess of caution, only touch the platform that is failing, for the current beta.

Jorg, I've pushed the version that touches all the platforms to C-C. If the L10N nightlies build with that, this patch can be uplifted to c-b for 57.
Assignee: nobody → mozilla
Flags: needinfo?(jorgk)
Attachment #8926098 - Flags: review+
OK, so will to trigger a Daily/Nightly now or wait for tomorrow's?
Flags: needinfo?(jorgk) → needinfo?(mozilla)
The nightly builds look good, so can be landed on comm-beta.
Flags: needinfo?(mozilla) → needinfo?(jorgk)
Attachment #8926095 - Flags: review+
You need to log in before you can comment on or make changes to this bug.