https://bugzilla.novell.com/show_bug.cgi?id=847971
https://bugzilla.novell.com/show_bug.cgi?id=847971#c13
--- Comment #13 from Raymond Wooninck 2013-11-27 13:07:25 UTC ---
Christian,
Thanks for checking, but did you really expected that I wouldn't have checked
that myself ?
I never said that Chromium needs the ffmpeg library from Packman. Chromium
itself contains the ffmpeg sources and builds in into the libffmpegsumo
library. Depending on the build dependencies, you would get either a
libffmpegsumo that just supports opensource codecs (like build on OBS) or one
with full support (like I am building on Packman)
Doing it your way, doesn't help as that the actual support for these codecs is
not in the ffmpegsumo library. Also leave 1 flag out doesn't help the
situation as that the build system, checks either one or the other, e.g. from
media/media.gyp:
['proprietary_codecs==1 or branding=="Chrome"', {
'sources': [
'mp2t/es_parser.h',
'mp2t/es_parser_adts.cc',
'mp2t/es_parser_adts.h',
'mp2t/es_parser_h264.cc',
'mp2t/es_parser_h264.h',
'mp2t/mp2t_common.h',
'mp2t/mp2t_stream_parser.cc',
'mp2t/mp2t_stream_parser.h',
'mp2t/ts_packet.cc',
'mp2t/ts_packet.h',
'mp2t/ts_section.h',
'mp2t/ts_section_pat.cc',
'mp2t/ts_section_pat.h',
'mp2t/ts_section_pes.cc',
'mp2t/ts_section_pes.h',
'mp2t/ts_section_pmt.cc',
'mp2t/ts_section_pmt.h',
'mp2t/ts_section_psi.cc',
'mp2t/ts_section_psi.h',
'mp4/aac.cc',
'mp4/aac.h',
'mp4/avc.cc',
'mp4/avc.h',
'mp4/box_definitions.cc',
'mp4/box_definitions.h',
'mp4/box_reader.cc',
'mp4/box_reader.h',
'mp4/cenc.cc',
'mp4/cenc.h',
'mp4/es_descriptor.cc',
'mp4/es_descriptor.h',
'mp4/mp4_stream_parser.cc',
And these indicated sources, are the one that is building the support for the
MP4 and H.264 stuff.
With regards to the changes between 12.3 and 12.3:Update, I can indicate the
following: The 12.3 package contains as well the flag
"-Dproprietary_codecs=1", however the source back then pushed all the support
into the ffmpegsumo library and things worked as we wanted it to work. The
Packman package was required to get the proprietary codec support and the OBS
package didn't even offer the libffmpegsumo.
However since then Google changed it where codecs were moved around and the
libffmpegsumo library is hardcoded required for Chromium to even start. Based
on that I already moved the package to Packman, but with the help of Ludwig
Nussel we were able to move it back as that OBS is allowed to have the ffmpeg
sources as long as the full package is not build.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.