Bruce Momjian wrote:
> > Zeugswetter Andreas DAZ SD wrote:
> > > 
> > > >> 1. the snprintf long long int check that uses int64 as variable name 
> > > >> which is 
> > > >>        a typedef in sys/inttypes.h
> > > >
> > > > I changed it to ac_int64.  Does that help?
> > > 
> > > One more is needed in c-library.m4.
> > 
> > OK, done.
> > 
> > > >> 2. whether strerror_r returns int check fails with a redefine of named 
> > > >> function 
> > > >>        from string.h
> > > >
> > > > I see.  Our test is:
> > > >         int strerror_r(int, char *, size_t);
> > > > but AIX is:
> > > >         int  strerror_r(int, char *, int);
> > > > OK, I added an #ifndef _AIX to use 'int' for the 3rd arg for AIX.
> > > 
> > > The wrong define is only on older AIX versions :-(
> > > And the patch produces wrong code asis (missing # and #endif).
> > > Since AIX always has the int returning function, how about no test on AIX:
> > > #ifndef_AIX
> > > int strerror_r(int, char *, size_t);
> > > #endif
> 
> OK, improved patch.  You mentioned only older versions of AIX have an
> 'int' for the 3rd strerror_r argument so we will just test for the
> return value on AIX and not the args:
> 
>       int strerror();
> 
> If there are other platforms that have this we could use it on all
> platforms but at this point we will do just AIX.

That is the perfect patch. Thank you Bruce. All tests pass on AIX 4.3.2 and 
5.2.0
with --enable-thread-safety and this last patch.

I am quite sure xlc_r is not needed to compile a thread safe libpq. Only those 
writing threaded clients will want to use it for compiling and linking their 
executables if they use usually not thread safe libc functions (like 
gethostbyname).

Andreas

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to