A small patch for building on Solaris and Illumos

Ian Collins ian at ianshome.com
Tue May 6 04:47:28 CEST 2014


Arran Cudbard-Bell wrote:
> On 23 Mar 2014, at 20:34, Ian Collins <ian at ianshome.com> wrote:
>
>> Hello,
>>
>> I would like to submit a small patch to enable clean building of 2.2.4 on Solaris 11 and Illumos.  Only two small changes are required:
>>
>> src/include/sysutmp.h:
>>
>> Line 17:
>> -#if !defined(sun) && !defined(sgi) && !defined(hpux)
>> +#if !defined(__sun) && !defined(sgi) && !defined(hpux)
> Out of interest what's the significant of __sun over sun or __sun__?

Hello again,

I finally got around to refreshing my build and I checked the v2 branch 
out of git (as described at http://git.freeradius.org/) and it looks 
like the symbol has been changed to _sun rather that __sun (note the 
double underscore).

>> src/include/libradius.h
>> After line 74:
>>
>> +#if defined(__sun)
>> +  typedef void(*sig_t)(int);
>> +#endif
>> +
> And that's probably handled better as an autoconf check.

Was this added anywhere? I can't see it.

-- 
Ian.



More information about the Freeradius-Devel mailing list