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)

defect
Not set
normal

Tracking

(firefox45 fixed)

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

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.
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
Blocks: 1203573
Could we not simply fix l10n-check to not overwrite the existing build?
Assignee: nobody → mh+mozilla
Component: General → Build Config
Product: Taskcluster → Core
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+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: