[rtmpdump] [PATCH] Make crypto support configurable through make variables

Howard Chu hyc at highlandsun.com
Wed Apr 7 09:40:22 CEST 2010


Martin Storsjö wrote:
> On Wed, 7 Apr 2010, Howard Chu wrote:
>
>> Martin Storsjö wrote:
>>> Hi,
>>>
>>> Currently, if one wants to build librtmp without crypto support, one has
>>> to edit rtmp.h to remove the CRYPTO definition and then edit the makefile
>>> to remove the reference to hashswf.o. With the attached patch, a user can
>>> simply do
>>>
>>> make CRYPTO=no
>>>
>>> to compile without crypto support, without having to edit the checked out
>>> files.
>>
>> You used Gmake-specific syntax in your patch, I've committed a generic
>> solution instead. Just use "make CRYPTO=" to turn off crypto support.
>
> Thanks, that looks very good!

This patch is still problematic. Removing CRYPTO from rtmp.h makes it 
impossible for an app using librtmp to determine whether CRYPTO support is 
present or not.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/


More information about the rtmpdump mailing list