Closed Bug 1836869 Opened 3 years ago Closed 3 years ago

Switch malloc_size_of_derive to syn 2/synstructure 0.13

Categories

(Core :: XPCOM, task)

task

Tracking

()

RESOLVED FIXED
116 Branch
Tracking Status
firefox116 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files)

No description provided.
Depends on: 1836229, 1836232
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/530d15a58ee6 Move malloc_size_of_derive as first-party into xpcom/rust. r=supply-chain-reviewers,emilio https://hg.mozilla.org/integration/autoland/rev/345be3af098b Switch malloc_size_of_derive to syn 2/synstructure 0.13. r=xpcom-reviewers,nika
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 116 Branch

I dunno how much people still care about the standalone WR build on github but this PR broke it, since it makes the WR workspace reference a file that's outside the gfx/wr folder and that isn't synced to github.

https://github.com/servo/webrender/pull/4818

Flags: needinfo?(jmuizelaar)
Flags: needinfo?(jmuizelaar)

I care about this not being broken. Mike, is there something we can do to fix it?

Flags: needinfo?(mh+mozilla)

Also for general awareness, I asked in the servo zulip chat and Martin Robinson said he'd try vendoring WR directly into servo. That was a few weeks ago and I don't know if there's been any update since. https://servo.zulipchat.com/#narrow/stream/263398-general/topic/Webrender.20sync

I guess we could use a patch in top-level Cargo.toml rather than a full path in the dependency.

Flags: needinfo?(mh+mozilla)

Can you put up a patch to do that?

Flags: needinfo?(mh+mozilla)

Can you file a new bug?

Flags: needinfo?(mh+mozilla) → needinfo?(jmuizelaar)
Flags: needinfo?(jmuizelaar)
Regressions: 1847139
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: