[PATCH v3 00/10] Solaris support

Tomi Ollila tomi.ollila at iki.fi
Thu Nov 15 07:55:14 PST 2012


On Wed, Nov 14 2012, Blake Jones wrote:

> Updated based on comments from Tomi Ollila last week:
>
> - Cleaned up the $(INSTALL) changes in vim/Makefile.
> - Fixed gen-version-script to be compliant with old "sh" implementation.

Ok -- tests on linux reveal...

Checking for strsep... No (will use our own instead).

and

Checking for standard version of asctime_r... No (will define
_POSIX_PTHREAD_SEMANTICS to get it).

testing...

$ gcc compat/have_strsep.c
compat/have_strsep.c: In function ‘main’:
compat/have_strsep.c:7:21: error: expected identifier or ‘(’ before ‘const’
compat/have_strsep.c:9:29: error: ‘delim’ undeclared (first use in this
function)
compat/have_strsep.c:9:29: note: each undeclared identifier is reported
only once for each function it appears in
zsh: exit 1     gcc compat/have_strsep.c


--- It is very easy to spot the problem ;)

$ gcc compat/check_asctime.c
compat/check_asctime.c: In function ‘main’:
compat/check_asctime.c:15:5: error: too many arguments to function
‘asctime_r’
In file included from compat/check_asctime.c:8:0:
/usr/include/time.h:266:14: note: declared here
zsh: exit 1     gcc compat/check_asctime.c

--- the posix-semantics way uses the 2-arg format. 

The logic of the test setting in this file doesn't open to
me. Why not test the same way as in getpwuid_r() case ?

Tomi


More information about the notmuch mailing list