Post-0.3 Android Sync Aurora uplift

RESOLVED FIXED

Status

Android Background Services
Android Sync
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: rnewman, Assigned: rnewman)

Tracking

unspecified
ARM
Android
Dependency tree / graph

Firefox Tracking Flags

(firefox11 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Gonna need a big snapshot. Preparing…
(Assignee)

Comment 1

6 years ago
A local build of the following queue produces a working Fennec 11.0a2 build on my machine.

The first item is a landing of Bug 710448, so we can skip over that.

  Bug 710448 - Rewritten Sync record reconciling to handle edge cases; r=rnewman, r=philikon, a=akeybl

The next five patches are the first preffed-off m-c landing. Some Makefile hackery was required to get these to apply.

  Bug 709391 - Part 1: implement in-Fennec Android Sync shipping solution. r=blassey, a=akeybl
  Bug 709391 - Part 2: l10n build changes to include Sync strings. r=blassey, f=Pike, a=akeybl
  Bug 709391 - Part 3: Makefile changes to preprocess sync_syncadapter.xml. r=blassey, a=akeybl
  Bug 709391 - initial import of Android Sync code (Git tag: v0.1.2-fixes). a=java-only, a=akeybl
  Bug 709391 - hide Sync setup task in Launcher, make syncs return immediately. a=java-only, a=akeybl

The next patch is some of Brad's Makefile work. Again, this needed a little effort to apply.

  Bug 712380 - Add support for clobbering the Android resources when needed. r=dougt, a=akeybl

The remaining patches are from today's 0.3 landing. I did a straight-up code drop to avoid any problems with missed files, rather than importing the patch I actually landed.

  Bug 709660 - correctly process resolution-specific Sync resources. r=mbrubeck, r=rnewman, a=akeybl
  Bug 718928 - AndroidManifest.xml does not depend on Sync manifest include fragments. r=ted, a=akeybl
  Bug 721271 - Post-0.3 Android Sync Aurora uplift. a=akeybl

Finally, there was some fixing-up needed in the final Makefile to get it to match more closely the m-c Makefile. Apparently hg continues to suck at merges, because our source dependencies weren't being built.

  Bug 721271 - Makefile fixes for Aurora landing. a=akeybl

I will attach the final Makefile tweaks for review, and as a place to hang an Aurora request.
(Assignee)

Comment 2

6 years ago
Created attachment 591686 [details] [diff] [review]
Makefile fixes for Aurora drop. v1

[Approval Request Comment]
User impact if declined: 
  No Sync.

Testing completed (on m-c, etc.): 
  Green build and tests on m-c.
  Manual testing of local Aurora build.
  QA of Try build of m-c drop.

Risk to taking this patch (and alternatives if risky):
  Delayed or missed Aurora uptake of Sync.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Attachment #591686 - Flags: approval-mozilla-aurora?

Comment 3

6 years ago
Comment on attachment 591686 [details] [diff] [review]
Makefile fixes for Aurora drop. v1

[Triage Comment]
Awesome - we're ready for our first sync drop on aurora! Approved
Attachment #591686 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 4

6 years ago
Thar she goes. Thanks Alex!

https://hg.mozilla.org/releases/mozilla-aurora/rev/76509146a8e5
https://hg.mozilla.org/releases/mozilla-aurora/rev/2bf61c2d0e29
https://hg.mozilla.org/releases/mozilla-aurora/rev/7b15cf9b89fa
https://hg.mozilla.org/releases/mozilla-aurora/rev/50911c719782
https://hg.mozilla.org/releases/mozilla-aurora/rev/4c20c43a09a0
https://hg.mozilla.org/releases/mozilla-aurora/rev/62cdb08ef4d3
https://hg.mozilla.org/releases/mozilla-aurora/rev/699a36336685
https://hg.mozilla.org/releases/mozilla-aurora/rev/2a12d4f824f4
https://hg.mozilla.org/releases/mozilla-aurora/rev/b314edef7cbf
https://hg.mozilla.org/releases/mozilla-aurora/rev/d786e7c31177
https://hg.mozilla.org/releases/mozilla-aurora/rev/b310cea4258c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
status-firefox11: --- → fixed
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Depends on: 716760
Component: Android Sync → Android Sync
Product: Mozilla Services → Android Background Services
You need to log in before you can comment on or make changes to this bug.