Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Figure out why mp4parse-capi is failing #19418

Closed
avadacatavra opened this issue Nov 29, 2017 · 3 comments
Closed

Figure out why mp4parse-capi is failing #19418

avadacatavra opened this issue Nov 29, 2017 · 3 comments
Labels
A-build Related to or part of the build process

Comments

@avadacatavra
Copy link
Contributor

#19413 fixes the problem but we should figure out why
@avadacatavra avadacatavra added the A-build Related to or part of the build process label Nov 29, 2017
@nox
Copy link
Contributor

nox commented Nov 29, 2017

I made mozilla/cbindgen#103 and filed mozilla/cbindgen#102.

@nox
Copy link
Contributor

nox commented Nov 29, 2017

The error we are talking about is:

     Running `/home/travis/build/servo/servo/target/debug/build/mp4parse_capi-7403969bf960d20d/build-script-build`

error: failed to run custom build command for `mp4parse_capi v0.9.1`

process didn't exit successfully: `/home/travis/build/servo/servo/target/debug/build/mp4parse_capi-7403969bf960d20d/build-script-build` (exit code: 101)

--- stdout

cargo:rerun-if-changed=src/lib.rs

--- stderr

thread 'main' panicked at 'Could not generate header: "couldn\'t execute `cargo metadata` with manifest \"/home/travis/build/servo/servo/.cargo/registry/src/github.com-1ecc6299db9ec823/mp4parse_capi-0.9.1/Cargo.toml\"."', /checkout/src/libcore/result.rs:906:4

@avadacatavra avadacatavra mentioned this issue Nov 29, 2017
Closed
nox added a commit that referenced this issue Nov 29, 2017
@nox
Update cbindgen to 0.3.2
f0ee599 
Should help us understand #19418.
@nox nox mentioned this issue Nov 29, 2017
Merged
bors-servo pushed a commit that referenced this issue Nov 29, 2017
Auto merge of #19423 - servo:bumps, r=jdm
f1fe370 
Update cbindgen to 0.3.2

Should help us understand #19418.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19423)
<!-- Reviewable:end -->
@ferjm
Copy link
Contributor

ferjm commented Dec 11, 2018

The mp4parse dependency was removed in #21543

@ferjm ferjm closed this as completed Dec 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-build Related to or part of the build process
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nox @ferjm @avadacatavra