On Sat, Jun 16, 2012 at 12:50:36PM +0200, Samuel Pitoiset wrote:
> This adds two protocols, but one of them is an internal implementation
> detail just used as an abstraction layer/generalization in the code. The
> RTMPT protocol implementation uses rtmphttp:// as an alternative to the
> tcp:// protocol. This allows moving most of the lower level logic out
> from the higher level generic rtmp code.
> ---
> - Fix missing proper dependecies in configure
> 
> --- a/configure
> +++ b/configure
> @@ -1511,6 +1511,10 @@ mmsh_protocol_select="http_protocol"
>  rtmp_protocol_deps="!librtmp_protocol"
>  rtmp_protocol_select="tcp_protocol"
> +rtmphttp_protocol_deps="!librtmp_protocol"
> +rtmphttp_protocol_select="http_protocol"
> +rtmpt_protocol_deps="!librtmp_protocol"
> +rtmpt_protocol_select="rtmphttp_protocol"
> --- a/libavformat/Makefile
> +++ b/libavformat/Makefile
> @@ -345,6 +345,7 @@ OBJS-$(CONFIG_MMST_PROTOCOL)             += mmst.o mms.o 
> asf.o
>  OBJS-$(CONFIG_RTMP_PROTOCOL)             += rtmpproto.o rtmppkt.o
> +OBJS-$(CONFIG_RTMPHTTP_PROTOCOL)         += rtmphttp.o rtmpproto.o rtmppkt.o
>  OBJS-$(CONFIG_RTP_PROTOCOL)              += rtpproto.o
> diff --git a/libavformat/allformats.c b/libavformat/allformats.c
> index 1320a28..69f27ab 100644
> --- a/libavformat/allformats.c
> +++ b/libavformat/allformats.c
> @@ -256,6 +256,8 @@ void av_register_all(void)
>      REGISTER_PROTOCOL (RTMP, rtmp);
> +    REGISTER_PROTOCOL (RTMPT, rtmpt);
> +    REGISTER_PROTOCOL (RTMPHTTP, rtmphttp);
>      REGISTER_PROTOCOL (RTP, rtp);

I'm not convinced dependencies are fixed properly; you seem to be missing
at least a Makefile entry for the rtmpt protocol.  Try

  configure --disable-everything --enable-protocol=rtmphttp
  configure --disable-everything --enable-protocol=rtmpt

from a clean tree and see if it compiles properly.

> --- a/doc/general.texi
> +++ b/doc/general.texi
> @@ -811,6 +811,7 @@ performance on systems without hardware floating point 
> support).
>  @item pipe         @tab X
>  @item RTMP         @tab X
> +@item RTMPT        @tab X
>  @item RTP          @tab X
>  @item TCP          @tab X

You will need to adapt this to my latest documentation commit.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to