This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: updated win32 macro


----- Original Message -----
From: "Akim Demaille" <akim@epita.fr>
To: "Robert Collins" <robert.collins@itdomain.com.au>
Cc: <cygwin-apps@sources.redhat.com>; <autoconf@gnu.org>
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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]