Vendor application-services into mozilla-central
Categories
(Firefox :: Sync, task)
Tracking
()
People
(Reporter: adw, Assigned: ttran)
References
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
65.62 KB,
patch
|
diannaS
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1859069 +++
Vendor application-services into mozilla-central so desktop can pick up this change to the suggest
component: https://github.com/mozilla/application-services/pull/5884
The PR hasn't been merged yet so I'm filing this bug as a placeholder.
Updated•8 months ago
|
Updated•8 months ago
|
Pushed by ttran@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/68f97e464090 Vendor application services into moz central r=lina,adw
Comment 4•8 months ago
|
||
bugherder |
Reporter | ||
Comment 5•8 months ago
|
||
There's a trivial conflict with the m-c patch on 120 in .cargo/config.in
.
Approval Request Comment
[Feature/Bug causing the regression]: This isn't a regression, it vendors in a new feature added to the cross-platform Firefox Suggest Rust component.
[User impact if declined]: This is required for the cross-platform Suggest experiment we plan to ship in 120.
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: Low risk
[Why is the change risky/not risky?]: This is only a vendor of Rust code that has its own tests in the application-services repo
[String changes made/needed]: None
Comment 6•7 months ago
|
||
Comment on attachment 9360714 [details] [diff] [review]
120 patch
Approved for 120.0b4
Updated•7 months ago
|
Description
•