Closed
Bug 1228289
Opened 9 years ago
Closed 9 years ago
Linux taskcluster builds have the x-test locale instead of en-US
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox45 fixed)
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
STR:
- Download a Linux buildbot build.
- Unpack it, start it.
- Open about:config
- Search for general.useragent.locale
- See that the value is en-US
- Download a Linux taskcluster build (on try, because all the links I try on inbound give me a "Artifact not found" error).
- Unpack it, start it.
- Open about:config
- Search for general.useragent.locale
- See that the value is x-test
This is all due to the use of MOZ_SIMPLE_PACKAGE_NAME, which triggers this snippet:
ifdef MOZ_SIMPLE_PACKAGE_NAME
PKG_BASENAME := $(MOZ_SIMPLE_PACKAGE_NAME)
else
PKG_BASENAME = $(MOZ_PKG_APPNAME)-$(MOZ_PKG_VERSION).$(AB_CD).$(MOZ_PKG_PLATFORM)
endif
Note how the PKG_BASENAME when MOZ_SIMPLE_PACKAGE_NAME is not set has AB_CD in it, which is the locale.
When l10n-check runs, which does a test repack, without MOZ_SIMPLE_PACKAGE_NAME, the en-US build is not overwritten since the filename for that test repack contains x-test instead of en-US. But with MOZ_SIMPLE_PACKAGE_NAME, l10n-check ends up overwriting target.tar.bz2, which leads to artifacts containing that test repack instead of the actual build.
Assignee | ||
Updated•9 years ago
|
Summary: Linux trycluster builds have the x-test locale instead of en-US → Linux taskcluster builds have the x-test locale instead of en-US
Comment 1•9 years ago
|
||
Comment 2•9 years ago
|
||
Could we not simply fix l10n-check to not overwrite the existing build?
Assignee | ||
Comment 3•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → mh+mozilla
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8692688 -
Flags: review?(ted)
Assignee | ||
Updated•9 years ago
|
Component: General → Build Config
Product: Taskcluster → Core
Comment 5•9 years ago
|
||
Comment on attachment 8692688 [details] [diff] [review]
Avoid l10n-check overwriting final package when MOZ_SIMPLE_PACKAGE_NAME is set
Review of attachment 8692688 [details] [diff] [review]:
-----------------------------------------------------------------
Simple but effective.
Attachment #8692688 -
Flags: review?(ted) → review+
Comment 7•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•