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)

enhancement

Tracking

(Not tracked)

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.