On Wed, Jul 18, 2012 at 3:45 PM, Diego Biurrun <[email protected]> wrote:
> On Tue, Jul 17, 2012 at 07:10:44PM +0300, Martin Storsjö wrote:
>> On Mon, 16 Jul 2012, Samuel Pitoiset wrote:
>> >--- a/configure
>> >+++ b/configure
>> >@@ -1526,6 +1528,9 @@ vfwcap_indev_extralibs="-lavicap32"
>> >
>> ># protocols
>> >+ffrtmpenc_protocol_deps="!librtmp_protocol"
>> >+ffrtmpenc_protocol_deps_any="gcrypt nettle openssl"
>> >+ffrtmpenc_protocol_select="tcp_protocol"
>> >httpproxy_protocol_deps="network"
>> >@@ -1541,6 +1546,7 @@ mmsh_protocol_select="http_protocol"
>> >rtmp_protocol_deps="!librtmp_protocol"
>> >rtmp_protocol_select="tcp_protocol"
>> >+rtmpe_protocol_select="ffrtmpenc_protocol"
>> >rtmphttp_protocol_deps="!librtmp_protocol"
>> >rtmphttp_protocol_select="http_protocol"
>> >rtmpt_protocol_deps="!librtmp_protocol"
>>
>> Since the others (rtmp and rtmpt) both have the
>> _deps="!librtmp_protocol", this one should as well, I think. In
>> principle it's not needed since we get the same dep via ffrtmpenc
>> though, but it's perhaps better to keep it for consistency.
>
> I think there is no need to repeat it.  Rather, drop it from the
> other protocols instead of duplicating.

Okay, I'll remove that and submit an other patch in order to this
useless duplication.

-- 
Best regards,
Samuel Pitoiset.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to