delorie.com/archives/browse.cgi | search |
----- Original Message ----- From: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au> To: "Akim Demaille" <akim AT epita DOT fr> Cc: <cygwin-apps AT sources DOT redhat DOT com>; <autoconf AT gnu DOT org> Sent: Thursday, March 15, 2001 4:18 AM Subject: Re: updated win32 macro > ----- Original Message ----- > From: "Akim Demaille" <akim AT epita DOT fr> > To: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au> > Cc: <cygwin-apps AT sources DOT redhat DOT com>; <autoconf AT gnu DOT 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. > > that's my suggestion as well. > 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. you don't lose any generality. what you gain is assurance that you are testing on a relevant platform. > 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. ps. you might consider extending it to handle optional arguments, a-la AC_PROG_CC_WIN32(HAVE_WIN32, [echo imma winnie], [echo imma no win]) implementation is left as an exercise to you =) > > Rob
webmaster | delorie software privacy |
Copyright © 2019 by DJ Delorie | Updated Jul 2019 |