Patrick Monnerat wrote:

> Your patch is OK for lib1900.c, but not for lib509.c because it uses
>
> (strdup)(ptr)
>
> However by changing your patch to
>
> #define strdup curlx_strdup
>
> It compiles fine with that.

Hi :-)

I don't foresee any problem in libtest programs if your #define was
used. Platforms which do have strdup wouldn't be affected.

> The next failure is linking: I do not export curlx_* symbols :-(

You would need to modify whichever packages/OS400 script is parsing
tests/libtest/Makefile.inc to generate your libtest/Makefile, in such
a way that '../../lib/strdup.c' is appended to lib509_SOURCES and
lib1900_SOURCES. You get the idea.

> Finally, I managed to enable strdup on OS400 (it exists, but only for
> C++: you have to define a special symbol to enable strcmpi(), strdup(),
> etc for C). I think it's the cleaner solution.

The above option would keep you in C territory ;-)

> By the way: if OS400 was the one an only platform without strdup(),
> would'nt it be time to suppress "HAVE_STRDUP" and assume it is always
> there ?

If memory serves me, long time ago Dan Fandrich also had some
platforms without strdup.

Cheers,

-- 
-=[Yang]=-
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html