Closed
Bug 792050
Opened 12 years ago
Closed 12 years ago
Allow to precompile startup cache without omnijar'ing first
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla18
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files)
6.03 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
2.37 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
This refactors precompile_cache.js to use the resource protocol handler to get the gre and app resource locations, and import all modules and components that can be found in either the corresponding omnijar or directory.
This will allow, in bug 780561, to avoid a two-step packaging, and in bug 755724, to properly precompile cache for both browser and metro without too much hassle.
Attachment #662166 -
Flags: review?(khuey)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → mh+mozilla
Assignee | ||
Comment 2•12 years ago
|
||
As a bonus, it also removes the import exceptions logs, which are confusing to many people looking at tinderbox logs.
Attachment #662166 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Assignee | ||
Comment 4•12 years ago
|
||
Backed out for pymake bustage.
https://hg.mozilla.org/integration/mozilla-inbound/rev/bf9c0d80b36e
Assignee | ||
Comment 5•12 years ago
|
||
Interdiff for a fixup to avoid the windows breakage. The problem was that MOZ_STARTUP_CACHE ended up with a path in the form e:/build/something, and NS_NewLocalFile barfs on these (because it expects absolute file names with backslashes).
Attachment #667369 -
Flags: review?(khuey)
Attachment #667369 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
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
•