[libav-devel] [PATCH] Fix httpproxy protocol dependencies

[Måns Rullgård]

John Stebbins <stebbins at jetheaddev.com> writes:

>> >>> +httpproxy_protocol_deps="http_protocol"
>> >> This is wrong.  I don't know what it should be, but I'm pretty sure it
>> >> should use a _select.
>> >>
>>
>> > Ok, we need the input of someone that understands how the
>> > dependencies work better then.  This did fix by build problem
>> > though.
>>
>> see mmsh_protocol_select just above =)
>
> I was hoping for an explanation why you would use one rather than the
> other since they both seem to "work".  But here's an updated patch.

Using _select makes things like --disable-protocols
--enable-protocol=httpproxy work as expected.

> From ba07b4888a5195ccb1161ae4edd5d64a5c4ffd00 Mon Sep 17 00:00:00 2001
> From: John Stebbins <jstebbins.hb at gmail.com>
> Date: Fri, 18 Nov 2011 13:39:18 -0800
> Subject: [PATCH] Fix httpproxy protocol dependencies
>
> Building with --disable-network failed with undefined reference.
> ---
>  configure |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/configure b/configure
> index 6e4259f..352150b 100755
> --- a/configure
> +++ b/configure
> @@ -1484,6 +1484,7 @@ http_protocol_deps="network"
>  http_protocol_select="tcp_protocol"
>  https_protocol_select="tls_protocol"
>  mmsh_protocol_select="http_protocol"
> +httpproxy_protocol_select="http_protocol"
>  mmst_protocol_deps="network"
>  rtmp_protocol_select="tcp_protocol"
>  rtp_protocol_select="udp_protocol"
> -- 

OK

-- 
Måns Rullgård
mans at mansr.com