NetBSD Problem Report #29577

From www@netbsd.org  Wed Mar  2 15:53:49 2005
Return-Path: <www@netbsd.org>
Received: by narn.netbsd.org (Postfix, from userid 31301)
	id CB6D063B117; Wed,  2 Mar 2005 15:53:49 +0000 (UTC)
Message-Id: <20050302155349.CB6D063B117@narn.netbsd.org>
Date: Wed,  2 Mar 2005 15:53:49 +0000 (UTC)
From: uwe@netbsd.org
Reply-To: uwe@netbsd.org
To: gnats-bugs@netbsd.org
Subject: AC_CHECK_FUNCS used incorrectly in tools/compat/configure.ac
X-Send-Pr-Version: www-1.0

>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:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Mar 02 15:54:00 +0000 2005
>Closed-Date:    Wed Mar 02 18:09:32 +0000 2005
>Last-Modified:  Wed Mar 02 18:11: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.

>Release-Note:

>Audit-Trail:

State-Changed-From-To: open->closed
State-Changed-By: wiz@netbsd.org
State-Changed-When: Wed, 02 Mar 2005 18:09:32 +0000
State-Changed-Why:
Fixed, thanks.


From: Thomas Klausner <wiz@netbsd.org>
To: gnats-bugs@netbsd.org
Cc: 
Subject: PR/29577 CVS commit: src/tools/compat
Date: Wed,  2 Mar 2005 18:08:45 +0000 (UTC)

 Module Name:	src
 Committed By:	wiz
 Date:		Wed Mar  2 18:08:45 UTC 2005

 Modified Files:
 	src/tools/compat: configure configure.ac

 Log Message:
 Remove commas in AC_CHECK_FUNCS that shouldn't be there.
 Closes PR 29577 by Valeriy E. Ushakov.


 To generate a diff of this commit:
 cvs rdiff -r1.50 -r1.51 src/tools/compat/configure
 cvs rdiff -r1.49 -r1.50 src/tools/compat/configure.ac

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

Home	PR Database Search