Commit: b386c3311d8c05ccbd075ab86be2ddace335b73b

boyang boyang at samba.org
Wed Dec 9 02:52:44 MST 2009


Björn JACKE wrote:
> Hi Bo,
>
>   
>> Thu Dec 10 09:22:25 2009
>> - ago - b386c3311d8c05ccbd075ab86be2ddace335b73b
>> show diffs
>> download diffs
>>
>> s3: Fix build on non-linux platform.
>> Signed-off-by: Bo Yang <boyang at samba.org>
>>     
>
> +/* PAM_RADIO_TYPE is linux-only. */
> +#ifdef LINUX
>
>
>
> any special reason why you used #ifdef LINUX instead of #ifdef PAM_RADIO_TYPE ?
>   
According to linux pam, PAM_RADIO_TYPE is linux specific. But I am not
sure whether PAM_RADIO_TYPE is a macro or not on other platforms...

If you are sure it is defined as macro, #ifdef LINUX should be equal to
#ifdef PAM_RADIO_TYPE.
> Cheers
> Björn
>   


-- 
Bo Yang, Software Engineer, Suse Labs
GPG-key-ID   538C4C1A
Samba Team   boyang at samba.org    http://www.samba.org/
SUSE Linux   boyang at suse.de      http://www.novell.com/



More information about the samba-technical mailing list