Closed Bug 1438449 Opened 6 years ago Closed 6 years ago

Remove unnecessary feature string in serde_derive patch

Categories

(Core :: Graphics: WebRender, enhancement, P3)

Other Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

https://searchfox.org/mozilla-central/rev/9011be0a172711bc243e50dfca16d42e877bf4ec/toolkit/library/rust/Cargo.toml#52 has a 'feature = "deserialize_in_place"' property which apparently is not used by cargo. The correct property name is 'features', not 'feature', and it takes an array of feature strings. But we're geting the desired behaviour even without specifying this feature so we can just drop it. This will eliminate the warning that cargo spits out during the build.
Comment on attachment 8951172 [details]
Bug 1438449 - Remove unused feature property from crate override.

https://reviewboard.mozilla.org/r/220438/#review226550
Attachment #8951172 - Flags: review?(jmuizelaar) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e5a696fe608
Remove unused feature property from crate override. r=jrmuizel
https://hg.mozilla.org/mozilla-central/rev/1e5a696fe608
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: