Closed
Bug 569355
Opened 15 years ago
Closed 15 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•15 years ago
|
Target Milestone: --- → 1.4
|
Assignee | |
Updated•15 years ago
|
Flags: blocking-fx-sync1.4+
|
|
Assignee | |
Updated•15 years ago
|
Assignee: mconnor → edilee
|
|
Assignee | |
Comment 2•15 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•15 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•15 years ago
|
Attachment #450241 -
Flags: review?(mconnor) → review+
|
|
Assignee | |
Comment 4•15 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: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 5•15 years ago
|
||
Attachment #452898 -
Flags: review?(mconnor)
|
|
Assignee | |
Comment 6•15 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•15 years ago
|
Attachment #452910 -
Flags: review?(mconnor)
You need to log in
before you can comment on or make changes to this bug.
Description
•