Closed
Bug 722127
Opened 13 years ago
Closed 13 years ago
Build --with-system-libvpx is broken for libvpx-1.0.0
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla12
People
(Reporter: ojab, Assigned: ojab)
References
()
Details
Attachments
(1 file, 1 obsolete file)
2.77 KB,
patch
|
derf
:
review+
|
Details | Diff | Splinter Review |
libvpx-1.0.0 is out (link to the changelog in URL), but firefox's configure fails to find system libvpx: $ make -f client.mk … checking for libvpx version >= v0.9.7... no configure: error: --with-system-libvpx requested but it is not v0.9.7 or later There is no VPX_CODEC_USE_INPUT_PARTITION define in libvpx-1.0.0 source tree.
Comment 1•13 years ago
|
||
Yeah, the API isn't quite as compatible as the CHANGELOG would suggest. I'll take care of this.
Assignee: nobody → tterribe
It seems that VPX_CODEC_USE_INPUT_PARTITION was changed to VPX_CODEC_USE_INPUT_FRAGMENTS in 1.0.0.
Attachment #592612 -
Flags: review?(tterribe)
Comment 3•13 years ago
|
||
Comment on attachment 592612 [details] [diff] [review] libvpx-1.0.0 has VPX_CODEC_USE_INPUT_FRAGMENTS define I think we actually want to drop support for v0.9.7, because of bug 696390. Just change the check to look for VPX_CODEC_USE_INPUT_FRAGMENTS only. That makes the comment above the check wrong (v1.0.0 does have new symbols we could check for, but they're encoder-only, and we currently don't require encode support). The version in the error message we give when v0.9.7 isn't found will also need to be updated.
Attachment #592612 -
Flags: review?(tterribe) → review-
Attachment #592622 -
Flags: review?(tterribe)
Updated•13 years ago
|
Attachment #592622 -
Flags: review?(tterribe) → review+
Keywords: checkin-needed
Comment 5•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/e73a68477cfd
Assignee: tterribe → ojab
Component: Video/Audio → Build Config
Keywords: checkin-needed
QA Contact: video.audio → build-config
Target Milestone: --- → mozilla12
Comment 6•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e73a68477cfd
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 7•13 years ago
|
||
(In reply to ojab from comment #2) > Created attachment 592612 [details] [diff] [review] > libvpx-1.0.0 has VPX_CODEC_USE_INPUT_FRAGMENTS define > > It seems that VPX_CODEC_USE_INPUT_PARTITION was changed to > VPX_CODEC_USE_INPUT_FRAGMENTS in 1.0.0. This makes no sense anymore, we could simple use the pkgconfig file to setup the libs/cflags even check version. This would make it much more robust and simply the code greatly.
Comment 8•13 years ago
|
||
(In reply to Jory A. Pratt from comment #7) > This makes no sense anymore, we could simple use the pkgconfig file to setup > the libs/cflags even check version. This would make it much more robust and > simply the code greatly. We'd probably take a patch to use pkgconfig. Feel free to open a new bug about it.
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•