Re: [exim] [exim-dev] Exim 4.80 RC1 uploaded

On 2012-05-20 at 00:20 +0200, Michael Haardt wrote:
> I tried RC1 on HP-UX 11.00 with gcc and apart from a few minor issues,
> things look fine.

Cool, thanks. :)

> I suggest not to use those functions:
>
> dcc.c: In function 'dcc_process':
> dcc.c:129: warning: incompatible implicit declaration of built-in function 'bzero'
> dcc.c:196: warning: incompatible implicit declaration of built-in function 'bcopy'

Those are the EXPERIMENTAL_DCC stuff, which is about to get a little
work from Wolfgang after the release, so I think it best to leave it.

> HP-UX 11 needs these additions to OS/os.h-HP-UX:
>
> ----------
> typedef struct __res_state *res_state;
>
> #define LLONG_MIN LONG_LONG_MIN
> #define LLONG_MAX LONG_LONG_MAX
>
> #define strtoll(a,b,c) strtoimax(a,b,c)
> ----------
>
> The first is needed due to the HP-UX resolver library. I don't know
> when res_state was defined, otherwise I would have used __RES to check
> if it needs to be defined. 19931104 does not have it yet.

*sigh* Thanks, I was worried this assumption would bite me; I'm
surprised directly messing with _res has proved so portable. I think I
saw some slightly struct names and want to avoid the main code depending
upon __doubled_underbar names, so I think that fixing this in the os.h
as you did is the correct path to take.

Only NetBSD has moved away from _res ... mind, their approach is a good
one and arguably correct.

Thanks for the fixes for HP-UX, pushed to master.
-Phil