Closed
Bug 1338655
Opened 8 years ago
Closed 8 years ago
mp4parse rust bindings no longer compile with cargo from Rust Nightly 2017-02-10
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: qdot, Assigned: rillian)
References
Details
Attachments
(1 file)
Due to a Cargo patch landing that now automatically includes build.rs files regardless of commands in Cargo.toml, the build.rs for mp4parse-capi is included in the build. Since the cheddar crate wasn't vendored in, we can't find it, and get a build error.
Reporter | ||
Updated•8 years ago
|
Summary: mp4parse rust bindings no longer compile under rust 1.17 (Rust Nightly 2017-02-10) → mp4parse rust bindings no longer compile with cargo from Rust Nightly 2017-02-10
Assignee | ||
Comment 1•8 years ago
|
||
Thanks for the report! Per irc discussion, now that we've vendored syntex, I'll just go ahead and vendor cheddar and stop patching out the build-depends line, assuming that doesn't confuse anything. Fallback is to patch out build.rs, since cargo is apparently auto-detecting it now.
Assignee: nobody → giles
Assignee | ||
Comment 2•8 years ago
|
||
This seems to be https://github.com/rust-lang/cargo/issues/3391 which suggests we can also patch in `build = false` in Cargo.toml. That would be a better solution if we aren't able to use the generated header.
Going forward, Ted suggested we hack (unmaintained) cheddar to use syn instead of syntex for the build-time speed advantage.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•8 years ago
|
||
We're not currently building syntex, so keeping the header generation at import time seems better here.
Comment 5•8 years ago
|
||
mozreview-review |
Comment on attachment 8836216 [details]
Bug 1338655 - Don't try to build mp4parse bindings.
https://reviewboard.mozilla.org/r/111682/#review113446
Attachment #8836216 -
Flags: review?(nfroyd) → review+
Pushed by rgiles@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/acfa7cfcb32b
Don't try to build mp4parse bindings. r=froydnj
Comment 7•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Comment 9•8 years ago
|
||
FYI, for me, it doesn't build with build=false, see bug 1340594.
Maybe the change you've landed depends on specific tool versions, and either the build should check for those versions, or somehow that file needs to be parameterized?
Updated•8 years ago
|
Depends on: 1340594
Comment 10•7 years ago
|
||
THUNDERBIRD_52_VERBRANCH:
https://hg.mozilla.org/releases/mozilla-esr52/rev/21af4e17cbf9f225e99c4103ac73a72311a702d1
Comment 11•7 years ago
|
||
(In reply to Jorg K (GMT+2) from comment #10)
> THUNDERBIRD_52_VERBRANCH:
> https://hg.mozilla.org/releases/mozilla-esr52/rev/
> 21af4e17cbf9f225e99c4103ac73a72311a702d1
It was backed out:
https://hg.mozilla.org/releases/mozilla-esr52/rev/c3c8a7968114
You need to log in
before you can comment on or make changes to this bug.
Description
•