diff mbox

[FFmpeg-devel] add hds demuxer

Message ID CADxeRwm9zScCMFsaHnpr1+sPf0y_cQc5tBb0gQsBeHqvWZGG+A@mail.gmail.com
State New
Headers show

Commit Message

Steven Liu Oct. 13, 2016, 2:15 p.m. UTC 
2016-10-13 17:41 GMT+08:00 Carl Eugen Hoyos <ceffmpeg@gmail.com>:

> 2016-10-13 8:51 GMT+02:00 Steven Liu <lingjiujianke@gmail.com>:
> > patch update.
> >
> > xml2 disabled by default
>
> Before reading Nicolas' mail, I wanted to suggest that you fix
> autodetection instead...
>
> Carl Eugen
>

Hi Carl,

      I need your help,
      I have search all the history mail, keywords 'autodetection' , i saw
your patch have autodetection context is like this:



Is this right? Or can you give me a doc link to learn this.

Comments

Carl Eugen Hoyos Oct. 14, 2016, 12:32 p.m. UTC | #1 
2016-10-13 16:15 GMT+02:00 Steven Liu <lingjiujianke@gmail.com>:

> Is this right? Or can you give me a doc link to learn this.

I am not really a "doc" type, more "trial-and-error", sorry.

You should - imo - take your original patch as you sent it,
temporarily move the libxml2 header on your system away
and make sure that FFmpeg still builds (without hds support).
The zmbv decoder would be an example of a component
that similarly depends on a systems library.

In any case, you have check wm4's suggestion and if you
don't like it, you have to explain why.

Carl Eugen
Steven Liu Oct. 14, 2016, 1:33 p.m. UTC | #2 
2016-10-14 20:32 GMT+08:00 Carl Eugen Hoyos <ceffmpeg@gmail.com>:

> 2016-10-13 16:15 GMT+02:00 Steven Liu <lingjiujianke@gmail.com>:
>
> > Is this right? Or can you give me a doc link to learn this.
>
> I am not really a "doc" type, more "trial-and-error", sorry.
>
Sorry for my abbreviations ， s/doc/documentation/g

>
> You should - imo - take your original patch as you sent it,
> temporarily move the libxml2 header on your system away
> and make sure that FFmpeg still builds (without hds support).
> The zmbv decoder would be an example of a component
> that similarly depends on a systems library.
>
patch will update.

>
> In any case, you have check wm4's suggestion and if you
> don't like it, you have to explain why.
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
diff mbox

Patch

diff --git a/libavformat/Makefile b/libavformat/Makefile
index 5d827d3..e2b4dd4 100644
--- a/libavformat/Makefile
+++ b/libavformat/Makefile
@@ -202,6 +202,7 @@  OBJS-$(CONFIG_H264_DEMUXER)              += h264dec.o
rawdec.o
 OBJS-$(CONFIG_H264_MUXER)                += rawenc.o
 OBJS-$(CONFIG_HASH_MUXER)                += hashenc.o
 OBJS-$(CONFIG_HDS_MUXER)                 += hdsenc.o
+OBJS-$(CONFIG_HDS_DEMUXER)               += hdsdec.o amfmetadata.o
f4mmanifest.o f4fbox.o flvtag.o
 OBJS-$(CONFIG_HEVC_DEMUXER)              += hevcdec.o rawdec.o
 OBJS-$(CONFIG_HEVC_MUXER)                += rawenc.o
 OBJS-$(CONFIG_HLS_DEMUXER)               += hls.o
diff --git a/libavformat/allformats.c b/libavformat/allformats.c
index 6a216ef..39505c3 100644
--- a/libavformat/allformats.c
+++ b/libavformat/allformats.c
@@ -146,9 +146,9 @@  void av_register_all(void)
     REGISTER_MUXDEMUX(H263,             h263);
     REGISTER_MUXDEMUX(H264,             h264);
     REGISTER_MUXER   (HASH,             hash);
-    REGISTER_MUXER   (HDS,              hds);
     REGISTER_MUXDEMUX(HEVC,             hevc);
     REGISTER_MUXDEMUX(HLS,              hls);
+    REGISTER_MUXDEMUX(HDS,              hds);
     REGISTER_DEMUXER (HNM,              hnm);
     REGISTER_MUXDEMUX(ICO,              ico);
     REGISTER_DEMUXER (IDCIN,            idcin);