Re: updated win32 macro

[Robert Collins]

----- Original Message -----
From: "Akim Demaille" <address@hidden>
To: "Robert Collins" <address@hidden>
Cc: <address@hidden>; <address@hidden>
Sent: Thursday, March 15, 2001 8:23 PM
Subject: Re: updated win32 macro


>
> | Sample code to use it in configure.in, when the program _needs_ the
> | win32 API:
> |
> | AC_CANONICAL_HOST
> |
> | case "${host}" in
> | *-*-cygwin*)
> |         AC_PROG_CC_WIN32
> |         if $ac_cc_win32; then
> |         dnl do nothing here - any header checks /library checks etc
> | later in configure.in will now pass
> |         else
> |             echo "configure: error: Win32 API needed and no
acceptable
> | cc could be found" 1>&2;
> |             exit 1;
> |         fi        ;;
> | esac
>
> My opinion is that AC_PROG_CC_WIN32 should contain an AC_REQUIRE of
> AC_CANONICAL_HOST, and should ensure the case $host itself.
>

Why? There is no side effect if it is tested for on platforms other than
cygwin. And by being a little bit more generic less changes will be
needed to work with (say) WINE. Or on a cross-compile chain.

And the developer writing the configure test will still need to decide
what to do if it fails && they are compiling on cygwin, so they still
need a case statement.

Rob