[Mesa-dev] [PATCH 1/2] build: add HAVE_SHA1 define when using --with-sha1 option

[Tapani Pälli]

On 09/03/2015 10:02 PM, Brian Paul wrote:
> On 09/03/2015 04:25 AM, Tapani Pälli wrote:
>> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
>> ---
>>   configure.ac | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 90ba4fe..0fed26e 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1110,6 +1110,11 @@ AC_MSG_RESULT([$with_sha1])
>>   AC_SUBST(SHA1_LIBS)
>>   AC_SUBST(SHA1_CFLAGS)
>>
>> +# Enable a define for SHA1
>> +if test "x$with_sha1" != "x"; then
>> +    DEFINES="$DEFINES -DHAVE_SHA1"
>> +fi
>> +
>>   # Allow user to configure out the shader-cache feature
>>   AC_ARG_ENABLE([shader-cache],
>>       AS_HELP_STRING([--disable-shader-cache], [Disable binary shader
>> cache]),
>>
>
> We should probably have similar code for scons too.

Yep, it seems options for sha1 (like many other configure.ac options) 
have not been added for scons build. I'll try to figure out how this 
feature could be enabled also for scons build.


> For this patch: Acked-by: Brian Paul <brianp at vmware.com>
>
> For patch 2: Reviewed-by: Brian Paul <brianp at vmware.com>
>
>