Closed Bug 1246592 Opened 7 years ago Closed 7 years ago

L10n repacks/installers are busted on windows for locales containing a dash


(Hello (Loop) :: Client, defect, P1)



(firefox45 affected, firefox46 fixed)

Tracking Status
firefox45 --- affected
firefox46 --- fixed


(Reporter: standard8, Assigned: standard8)



(Whiteboard: [merge after/at same time as bug 1239828])


(2 files, 1 obsolete file)

Following the landing of bug 1239828, L10n repacks are busted on Windows - AFAICT for only locales that have a dash in the locale name:

IOError: [Errno 2] No such file or directory: '../../../dist/xpi-stage/locale-nb-NO/browser\\features/\\chrome\\content/shared/vendor/sdk-content/images/rtc/access-denied-copy-firefox.png'
Seems that weirdo heuristic to use shell or not when executing python process? I recall another bug recently where quoting has helped.
I've tried reproducing this locally on my own system, but it runs successfully there.

I'm using:

./mach configure
<copy package/installer files into dist>
cd objdir/browser/locales
compare-locales -m /e/merged/ /e/fx-team/browser/locales/l10n.ini /e/fx-team/l10n-central/ en-GB
mozmake.exe langpack-en-GB LOCALE_MERGEDIR=/e/merged
mozmake.exe installers-en-GB LOCALE_MERGEDIR/e/merged
Mike, can you help us here, not being able to reproduce is making it harder for me to guess where we might have got things wrong...
Flags: needinfo?(mh+mozilla)
The best thing to do would be to separate out locales in a separate and change the path that browser/locales recurses into. It's actually not a good thing that content and locales are mixed up, that means we're shipping content in langpacks.
Flags: needinfo?(mh+mozilla)
Thank you for the suggestion Mike. This does the separation and I've tested it locally on Mac to check it works.

I couldn't reproduce the windows changes, so I'm not sure its worth testing there.
Attachment #8717336 - Flags: review?(mh+mozilla)
Attachment #8717336 - Flags: review?(mh+mozilla) → review+ looks good now.

Thank you for the hint Mike.
Assignee: nobody → standard8
Closed: 7 years ago
Rank: 10
Component: Build Config → Client
Priority: -- → P1
Product: Core → Hello (Loop)
Resolution: --- → FIXED
Version: 33 Branch → unspecified
Comment on attachment 8717336 [details] [diff] [review]
Try and fix windows locale repack bustage by adjusting how locales are built for Loop.

Approval Request Comment
[Feature/regressing bug #]: Bug 1239828
[User impact if declined]: Broken L10n repacks on Windows for locales with a dash.
[Describe test coverage new/current, TreeHerder]: Landed in m-c. Passes L10n builds
[Risks and why]: Low, simple build adjustment to bug 1239828
[String/UUID change made/needed]: None
Attachment #8717336 - Flags: approval-mozilla-aurora?
Attached patch Patch for beta (obsolete) — Splinter Review
Approval Request Comment

Beta version, please see comment 8 for details.
Attachment #8717546 - Flags: approval-mozilla-beta?
Attached patch Patch for betaSplinter Review
Approval Request Comment

Correct patch for beta.
Attachment #8717546 - Attachment is obsolete: true
Attachment #8717546 - Flags: approval-mozilla-beta?
Attachment #8717559 - Flags: approval-mozilla-beta?
Whiteboard: [merge after/at same time as bug 1239828]
Blocks: 1247424
Comment on attachment 8717336 [details] [diff] [review]
Try and fix windows locale repack bustage by adjusting how locales are built for Loop.

Locale build fix for Hello, ok to uplift to aurora.
Attachment #8717336 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 8717559 [details] [diff] [review]
Patch for beta

Should be in 45 beta 6.
Attachment #8717559 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: needinfo?(standard8)
backed out by request from sylvestre in
Flags: needinfo?(standard8)
Flags: needinfo?(standard8)
You need to log in before you can comment on or make changes to this bug.