Closed Bug 1215696 Opened 5 years ago Closed 5 years ago
update rust mp4parser to v0
Kinetik's fix for only parsing the 'ftyp' box.
Good catch figuring out the 'zero tracks' thing. Thanks!
Attachment #8675154 - Flags: review?(kinetik)
We give the correct answer to the 'street.mp4' test now.
Attachment #8675156 - Flags: review?(kinetik)
Attachment #8675154 - Flags: review?(kinetik) → review+
Attachment #8675156 - Flags: review?(kinetik) → review+
BTW: remember to update Cargo.toml to 0.1.1.
Backed out for build bustage: https://hg.mozilla.org/integration/mozilla-inbound/rev/ebad4f2326b0 https://treeherder.mozilla.org/logviewer.html#?job_id=15886494&repo=mozilla-inbound
Hrm, it happened a second time after my backout. Maybe it's some clobber-needed thing?
Trying again with clobber.
Looks like the clobber did the trick. We don't have proper dependency generation for rust (patches pending) but I didn't expect that to be a problem because the top-level crate source changed. Strange.
(In reply to Matthew Gregan [:kinetik] from comment #3) > BTW: remember to update Cargo.toml to 0.1.1. Thanks, I forgot to do that, and then couldn't upload the package on Friday anyway because of https://github.com/rust-lang/cargo/issues/2063. I moved the tag and pushed the changes yesterday. https://crates.io/crates/mp4parse/0.1.1
4 years ago
Depends on: 1215234
4 years ago
You need to log in before you can comment on or make changes to this bug.