bug-mailutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: compiling on Nto...


From: Alain Magloire
Subject: Re: compiling on Nto...
Date: Sat, 14 Apr 2001 01:11:38 -0400 (EDT)

> 
> ChangeLog:
> 
>   * QNX needed to include <strings.h> for many of the useful string functions.

Ok,
Which one was POSIX/ANSI C again? <string.h> or <strings.h> ?
I think it is <string.h>,  <strings.h> is the obsolete one and bet
that strcasecmp which is not part of POSIX is define in <strings.h>

>   * QNX needed to define _QNX_SOURCE so <sys/time.h> doesn't have an internal
>     warning.

Ok.

>   * mailbox/mime.c: wasn't including config.h

Oops.

>   * mailbox/attachment.c, configure.in: QNX needed libgen.h for basename().

Ok.
Another POSIX thing. basename() is not posix.  But I thing the latest
UNIX98 says it can be in libgen.h.

>   * pop3d/user.c: QNX says getspnam(char* n), I cast away the error.
> 

QNX is wrong is should be "const char *".

> I don't know about that last, but I thought I'd throw it in, should work
> even it getspnam(const char*) is the prototype, I can't imagine it's
> actually writing to the name argument.

Ok.

> 
> And what on earth is vasprintf? I've never heard of it, the imap code
> might want to check for it's existence before calling it. It looks
> strikingly similar to vsprintf, from it's usage in imap4d/util.c,
> util_send()/util_out(). That's a link time problem for me.

Blame Jakob ;-), for to many GNUismes.
vasprintf () is a GNU extension, it is the same
as vsprintf expet the buffer is allocate/grow on demand.

I'll check in a patch for it later.

One more: argp.h
mailutils/mailx will not compile on non GNU plaforms because of argp.
Before you ask, argp is another GNU extension.  It gives a very
nice user level API instead of getopt() or getopt_long() when
trying to be fancy with options.  Unfortunately, argp is not
portable(well it is but we'll take some work).

> Still don't know what's up with automake, looks like an installation
> problem.

Maybe automake was not install correctly.

You should have a version of mdate-sh in
${prefix}/share/automake/mdate-sh
On my linux box it will map to
/usr/local/share/automake/mdate-sh

> 
> Good night,

Thanks.
Will look at the patch later.

--
alain




reply via email to

[Prev in Thread] Current Thread [Next in Thread]