Re: [PATCH] autoconf: Add test for sys/select.h header file

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 25, 2008, 12:30 UTC
Message-ID: <alpine.LSU.1.00.0801251229250.5731@racer.site>
In-Reply-To: <1201259981-7115-1-git-send-email-jnareb@gmail.com>

Hi,

On Fri, 25 Jan 2008, Jakub Narebski wrote:

Show 14 quoted lines

> diff --git a/configure.ac b/configure.ac
> index af177fd..85d7ef5 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -235,6 +235,12 @@ test -n "$NEEDS_SOCKET" && LIBS="$LIBS -lsocket"
>  ## Checks for header files.
>  AC_MSG_NOTICE([CHECKS for header files])
>  #
> +# Define NO_SYS_SELECT_H if you don't have sys/select.h.
> +AC_CHECK_HEADER([sys/select.h],
> +[NO_SYS_SELECT_H=],
> +[NO_SYS_SELECT_H=UnfortunatelyYes])
> +AC_SUBST(NO_SYS_SELECT_H)
> +#

Just because I am curious: would that not define "NO_SYS_SELECT_H" in both cases? IOW would the "ifdef NO_SYS_SELECT_H" not be triggered all the time?

Thanks, Dscho