Closed
Bug 569355
Opened 14 years ago
Closed 14 years ago
source code reorg, redux
Categories
(Cloud Services :: General, defect)
Cloud Services
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.4
People
(Reporter: mconnor, Assigned: Mardak)
References
()
Details
Attachments
(2 files, 2 obsolete files)
95.41 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
12.62 KB,
patch
|
Details | Diff | Splinter Review |
Goals: * Separate crypto and sync modules cleanly from UI pieces * Split Fennec/Firefox UI/strings/etc * Make it possible to hg convert a subset into something we can push into mozilla-central at controlled code drop points.
Reporter | ||
Updated•14 years ago
|
Target Milestone: --- → 1.4
Assignee | ||
Updated•14 years ago
|
Flags: blocking-fx-sync1.4+
Assignee | ||
Updated•14 years ago
|
Assignee: mconnor → edilee
Assignee | ||
Comment 2•14 years ago
|
||
Main oddness is strings loaded from util.js into Str but most aren't actually used by the service. Some files are duplicated like sync-32x32.png as it's used for the add-on icon as well as other app-specific UI bits.
Attachment #450229 -
Flags: review?(mconnor)
Assignee | ||
Comment 3•14 years ago
|
||
test: build - $(MAKE) -k -C tests/unit + $(MAKE) -k -C services/sync/tests/unit
Attachment #450229 -
Attachment is obsolete: true
Attachment #450241 -
Flags: review?(mconnor)
Attachment #450229 -
Flags: review?(mconnor)
Reporter | ||
Updated•14 years ago
|
Attachment #450241 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 4•14 years ago
|
||
http://hg.mozilla.org/services/fx-sync/rev/f5a64d0ecc66 Split the code structure to service/{crypto,sync} and ui/{fennec,firefox}. Update the top level makefile to build the addon staging directory to merge various parts of directories.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•14 years ago
|
||
Attachment #452898 -
Flags: review?(mconnor)
Assignee | ||
Comment 6•14 years ago
|
||
Fennec doesn't sync prefs so no need to copy there.
Attachment #452898 -
Attachment is obsolete: true
Attachment #452910 -
Flags: review?(mconnor)
Attachment #452898 -
Flags: review?(mconnor)
Assignee | ||
Updated•14 years ago
|
Attachment #452910 -
Flags: review?(mconnor)
You need to log in
before you can comment on or make changes to this bug.
Description
•