Closed Bug 1373317 Opened 7 years ago Closed 7 years ago

stylo: Bindgen upgrade busted mac builds due to prepended underscore

Categories

(Firefox Build System :: General, enhancement, P1)

Product:
Firefox Build System
Component:
General
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bholley, Assigned: emilio)

References

Details

After a recent pull, mac won't link: https://gist.github.com/bholley/d0c35f1cad08efe0c8fa75c40ba6ac91 This is because bindings now include: extern "C" { #[link_name = "_Gecko_MaybeCreateStyleChildrenIterator"] pub fn Gecko_MaybeCreateStyleChildrenIterator(node: RawGeckoNodeBorrowed) -> StyleChildrenIteratorOwnedOrNull; } Where there was previously no link_name specified. I suspect https://github.com/servo/rust-bindgen/pull/729 , which landed in a bindgen update recently. As a hacky workaround, you can do: perl -ni.bak -e 'print unless /link_name/' PATH/TO/GENERATED/bindings.rs But this obviously needs to be redone every time gecko headers change, which isn't great. We should fix this as soon as possible.
Priority: -- → P1
Flags: needinfo?(nfitzgerald)
Flags: needinfo?(emilio+bugs)
https://github.com/servo/rust-bindgen/pull/752 and https://github.com/servo/servo/pull/17341
Assignee: nobody → emilio+bugs
Flags: needinfo?(nfitzgerald)
Flags: needinfo?(emilio+bugs)
Blocks: stylo-nightly-build
No longer blocks: stylo
Both landed.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1375774
No longer blocks: stylo-nightly-build
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.