[PATCH][SAMBA4] 3 patches to get samba4 to build on OpenSolaris 08.11

[David Collier-Brown]

Torgeir Lerkerød wrote:
>>> From how I read it, it just checks for options that happen to be for
>>> the IRIX CC - and that it will not add any CFLAGS that don't exist.
>>> Does this particular test fail on OpenSolaris for some reason?
>>>       
>> Becase the cc gives something else back. Check SunX on the build farm to
>>     
> see
>   
>> the same thing.
>>     
>
>   
>> sunX doesn't build with --enable-developer, so doesn't hit this problem.
>> What does this option do exactly that breaks stuff on OpenSolaris? Are
>> the characters in -fullwarn interpreted as individual options? What
>> breaks?
>>     
>
> It's interperted as an invalid option.
>
>
>   
>>>>>> 0002-Fix-include.patch:
>>>>>>  This should ideally go into heimdal upstream, as the code is
>>>>>>  imported from there.
>>>>>>             
>>>>> Well it wouldn't have been a problem if samba 4's heimdal version
>>>>>           
>> wasn't
>>     
>>>>> from aug 2008. There are other fixes like this in there done by
>>>>>           
>> metze. So
>>     
>>>>> this needs to go in to build on Solaris until somebody do an upgrade
>>>>>           
>> of
>>     
>>>>> heimdal... Of'course you can ignor Solaris as a platform, but if
>>>>>           
>> not....
>>     
>>>> I'm quite sure we've imported changes from Heimdal since august, and
>>>>         
>> the
>>     
>>>> diff with Heimdal is quite small these days.
>>>>         
>>> The date was 28 of aug 2008. You can find the tree that went in her :
>>>
>>>       
>> http://gitweb.samba.org/?p=metze/heimdal/wip.git;a=shortlog;h=refs/heads/lorikeet-heimdal
>>     
>>> .
>>> And as a reference there are numberous commits by metze fixing things
>>>       
>> like
>>     
>>> this in source4/heimdal from about 2 weeks back to get samba 4 to build
>>>       
>> on
>>     
>>> other systems....
>>>       
>> The last import was from 27 October,
>> 2b29b7186459d945ec448694164bfe4239b30d72 and
>>
>> http://git.samba.org/?p=metze/heimdal/wip.git;a=shortlog;h=refs/heads/lorikeet-heimdal-200810271034
>>
>> I wasn't aware we still had this many changes against Heimdal, I'll put
>> it in.
>>
>> Unfortunately the patch I already committed broke some other hosts, so I
>> had to revert it. Does cpp on Solaris assume that all input files are C
>> files, or is there some option to specify the file type?
>>     
>
>
> Hmm... I swear I tested it... Sorry about that. I can only check with the
> compiler I use, which is the november edition of Sun Studio. Sun cc doesn't
> need a flag as it defaults to -Xa which is Sun version of "Assume ANSI C
> complince", which also by default enables -xc99.
>   

The Sun compiler driver initially looks at file-extensions to decide if
you just passed
it C, C++ or Fortran (;-)). By default cc expects C code and sets the
defaults
to CPP and the parser to suit ANSI C.

> Regards,
> Torgeir
>
>   


-- 
David Collier-Brown,         | Always do right. This will gratify
System Programmer and Author | some people and astonish the rest
davecb at spamcop.net           |                      -- Mark Twain
(416) 223-8968