[Devel] openser on Solaris

[Wolfgang Hottgenroth]

Hi Bogdan,


Bogdan-Andrei Iancu wrote:
> Hi Wolfgang,
>
> the IPTOS_MINCOST issue is already fixed on the CVS, so please update
> and see if it works.
>
Okay.


> the fix for the "sed" problem was committed on CVS - thanks!
>
You're welcome - and thanks for openser ;-)

> regarding the C++ stuff, it will not be included on the CVS since
> there is no C++ module on the cvs ;).
>
Well, isn't the module-concept an invitation for people to write their
own modules, even those which might not be good enough or too specific
to publish them? I would really appreciate to have these rules in the
both files, as the need to patch openser here would disappear
completely. Especially as these rules doesn't hurt. Or do they?


Cheers,
Wolfgang



> Regards,
> Bogdan
>
> Wolfgang Hottgenroth wrote:
>
>> Hi,
>>
>>
>>
>> I had some problems to compile openser 1.1.0 on Solaris.
>>
>> 1.) Solaris sed does not support -i (edit in place). This happens in the
>> Makefile when installing openserctl. I've changed the Makefile.
>> 2.) Solaris has no IPTOS_MINCOST. There is already a cpp logic in cfg.y
>> which should disable this on Solaris, but this logic seems to be buggy:
>>
>> #if !defined(__OS_solaris) || !defined(__OS_netbsd)
>> } else if (strcasecmp($3,"IPTOS_MINCOST")) {
>> tos=IPTOS_MINCOST;
>> #endif
>>
>> Certainly #if !defined(__OS_solaris) && !defined(__OS_netbsd) or #if
>> !(defined(__OS_solaris) || defined(__OS_netbsd)) (de Morgane's rule?)
>> was meant. Otherwise IPTOS_MINCOST is only remove if the OS is both
>> Solaris and NetBSD ;-)
>> If changed cfg.y.
>>
>> Additionally I've changed Makefule.rules and Makefile.sources to make
>> C++ modules possible.
>>
>> I've attached an unifed patch.
>>
>>
>>
>> Cheers,
>> Wolfgang
>>
>>
>