only package webapp runtime l10n files when the runtime is enabled

RESOLVED FIXED in Firefox 15

Status

Firefox Graveyard
Webapp Runtime
P3
normal
RESOLVED FIXED
5 years ago
a year ago

People

(Reporter: myk, Assigned: myk)

Tracking

14 Branch
Firefox 15
Bug Flags:
in-moztrap -

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 628094 [details] [diff] [review]
patch v1: the obvious fix

We currently package the webapp runtime's l10n files unconditionally, even if the runtime is disabled (f.e. on Linux at the moment).  We should do so only when the runtime is enabled.
Attachment #628094 - Flags: review?(benjamin)
(In reply to Myk Melez [:myk] [@mykmelez] from comment #0)
> Created attachment 628094 [details] [diff] [review]
> patch v1: the obvious fix
> 
> We currently package the webapp runtime's l10n files unconditionally, even
> if the runtime is disabled (f.e. on Linux at the moment).  We should do so
> only when the runtime is enabled.

Could this same logic of this patch be applied to bug 706634?
Mean to say - similar use of that "MOZ_WEBAPP_RUNTIME" parameter
(Assignee)

Comment 3

5 years ago
Not sure, but whoever takes on bug 706634 will be qualified to make that judgement.
(Assignee)

Updated

5 years ago
Priority: -- → P3
Attachment #628094 - Flags: review?(benjamin) → review+

Updated

5 years ago
Whiteboard: [qa-]
https://hg.mozilla.org/mozilla-central/rev/2f9736a30755
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Flags: in-moztrap-

Updated

5 years ago
QA Contact: jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.