PATCH: no ac_cv_cygwin in 2.50

PATCH: no ac_cv_cygwin in 2.50

[Andrej Borsenkow]

It is disappeared; canonical way is to use $host_os. Hmm ... should it be
$target_os, really? Anybody tried to cross-compile zsh?

-andrej

Index: zshconfig.ac
===================================================================
RCS file: /cvsroot/zsh/zsh/zshconfig.ac,v
retrieving revision 1.1
diff -u -r1.1 zshconfig.ac
--- zshconfig.ac        2001/06/06 08:37:21     1.1
+++ zshconfig.ac        2001/06/06 11:29:57
@@ -1538,7 +1538,7 @@
   zsh_cv_sys_dynamic_strip_exe="${zsh_cv_sys_dynamic_strip_exe=yes}"
   zsh_cv_sys_dynamic_strip_lib="${zsh_cv_sys_dynamic_strip_lib=yes}"
   zsh_cv_sys_dynamic_broken="${zsh_cv_sys_dynamic_broken=no}"
-elif test "x$ac_cv_cygwin" = xyes; then
+elif test "$host_os" = cygwin; then
   DL_EXT="${DL_EXT=dll}"
   DLLD="${DLLD=dllwrap}"
   DLLDFLAGS="${DLLDFLAGS=--export-all-symbols}"
@@ -1738,7 +1738,7 @@
   if $strip_libldflags && test "$zsh_cv_sys_dynamic_strip_lib" = yes; then
     LIBLDFLAGS="$LIBLDFLAGS -s"
   fi
-  if test "x$ac_cv_cygwin" = xyes; then
+  if test "$host_os" = cygwin; then
     INSTLIB="install.cygwin-lib"
     UNINSTLIB="uninstall.cygwin-lib"
   fi
@@ -1772,7 +1772,7 @@
 AC_SUBST(INSTLIB)dnl
 AC_SUBST(UNINSTLIB)dnl

-if test x"$ac_cv_cygwin" = xyes; then
+if test "$host_os" = cygwin; then
   EXTRAZSHOBJS="$EXTRAZSHOBJS zsh.res.o"
 fi

Re: PATCH: no ac_cv_cygwin in 2.50

[Alexandre Duret-Lutz]

>>> "Andrej" == Andrej Borsenkow <Andrej.Borsenkow@mow.siemens.ru> writes:

 Andrej> It is disappeared; canonical way is to use
 Andrej> $host_os. Hmm ... should it be $target_os, really?

No.  Any use of $target would be wrong in Zsh.  

The handling of --build, --host, and --target was a real mess in
autoconf =< 2.13, but now it's crystal clear:

  --build  is the arch on which you build,
  --host   is the arch on which the program will run
  --target is used *only* when building compiler tool-chains and specifies the
           arch for which they should produce output.

There are A LOT of configure.in out there that erroneously use
$target in lieu of $host.

[...]
-- 
Alexandre Duret-Lutz