Subject: toolchain/29577: AC_CHECK_FUNCS used incorrectly in tools/compat/configure.ac
To: None <toolchain-manager@netbsd.org, gnats-admin@netbsd.org,>
From: None <uwe@netbsd.org>
List: netbsd-bugs
Date: 03/02/2005 15:54:00
>Number:         29577
>Category:       toolchain
>Synopsis:       AC_CHECK_FUNCS used incorrectly in tools/compat/configure.ac
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    toolchain-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Mar 02 15:54:00 +0000 2005
>Originator:     Valeriy E. Ushakov
>Release:        2.99.16
>Organization:
>Environment:
FreeBSD sampo 4.10-STABLE FreeBSD 4.10-STABLE #0: Thu Nov 18 02:46:25 MSK 2004     uwe@sampo:/usr/obj/usr/src/sys/SAMPO  i386
>Description:
There's a number of AC_CHECK_FUNCS instances with incorrect
list of functions to check, e.g.:

AC_CHECK_FUNCS([bswap16, bswap32, bswap64] ...)

The list of functions should be space separated, not comma separated.
This error results in wrong configure tests.

checking for bswap16,... no
checking for bswap32,... no
checking for bswap64... no

The C code for the test uses incorrect name with the trailing comma
all over the place, e.g:

char bswap16, ();

>How-To-Repeat:
Check $(OBJDIR)/tools/compat/config.log of any build.

>Fix:
Drop those commas.