Closed
Bug 1469228
Opened 6 years ago
Closed 6 years ago
Upgrade rust libraries to not use syn-0.12
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox62 fixed)
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: kats, Assigned: emilio)
References
Details
As a result of servo/webrender#2819, mozilla-central is going to start dragging in syn 0.14. Right now third_party/rust has syn 0.13 and syn 0.12. We should upgrade stuff such that we can drop syn 0.12, so that we don't end up with three copies of syn in the tree.
Comment 1•6 years ago
|
||
`cargo update -p cstr-macros` removes one dependency. The rest should be taken care of by upgrading malloc_size_of_derive and style_derive from https://github.com/servo/servo/, they have been upgraded to syn 0.13 there. I think Emilio has a script for cherry-picking across repos.
Flags: needinfo?(emilio)
Assignee | ||
Comment 2•6 years ago
|
||
Yup, happy to take this.
Assignee: nobody → emilio
Flags: needinfo?(emilio)
Pushed by ecoal95@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/1ffb899a5f02 Cherry-pick malloc_size_of_derive and style_derive changes from servo/servo#20690. r=me https://hg.mozilla.org/integration/mozilla-inbound/rev/5ea92f9e5043 Update cstr-macros to v0.1.3. r=me https://hg.mozilla.org/integration/mozilla-inbound/rev/f3a4d5ac04be Revendor rust dependencies. r=me
Reporter | ||
Comment 4•6 years ago
|
||
Thanks!
Comment 5•6 years ago
|
||
https://github.com/servo/servo/issues/21059 tracks moving Servo and its dependencies to 0.14 (removing 0.13). The dependency graphs is different from Gecko’s, but has significant overlap.
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1ffb899a5f02 https://hg.mozilla.org/mozilla-central/rev/5ea92f9e5043 https://hg.mozilla.org/mozilla-central/rev/f3a4d5ac04be
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•