Closed Bug 759489 Opened 9 years ago Closed 9 years ago

only package webapp runtime l10n files when the runtime is enabled

Categories

(Firefox Graveyard :: Webapp Runtime, defect, P3)

14 Branch
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 15

People

(Reporter: myk, Assigned: myk)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

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
Not sure, but whoever takes on bug 706634 will be qualified to make that judgement.
Priority: -- → P3
Attachment #628094 - Flags: review?(benjamin) → review+
Whiteboard: [qa-]
https://hg.mozilla.org/mozilla-central/rev/2f9736a30755
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Flags: in-moztrap-
QA Contact: jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.