delorie.com/archives/browse.cgi   search  
Mail Archives: cygwin-developers/2000/03/30/22:46:49

Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT sourceware DOT cygnus DOT com>
List-Archive: <http://sourceware.cygnus.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT sourceware DOT cygnus DOT com>
List-Help: <mailto:cygwin-developers-help AT sourceware DOT cygnus DOT com>, <http://sourceware.cygnus.com/ml/#faqs>
Sender: cygwin-developers-owner AT sourceware DOT cygnus DOT com
Delivered-To: mailing list cygwin-developers AT sourceware DOT cygnus DOT com
X-Authentication-Warning: hp2.xraylith.wisc.edu: khan owned process doing -bs
Date: Thu, 30 Mar 2000 21:45:59 -0600 (CST)
From: Mumit Khan <khan AT NanoTech DOT Wisc DOT EDU>
To: cygwin-developers AT sourceware DOT cygnus DOT com
Subject: Re: net release/installer status?
In-Reply-To: <20000328174122.A29443@cygnus.com>
Message-ID: <Pine.HPP.3.96.1000330213918.1711A-100000@hp2.xraylith.wisc.edu>
MIME-Version: 1.0

On Tue, 28 Mar 2000, Chris Faylor wrote:

> 
> I don't see a syntax error.  It seems to be legal to have:
> 
> 	for f ; in stuff; do
> 
> I don't know why you're seeing an error.  Is the above command an error
> on your system?

Same here. Bash, pdksh, ash, the real ksh from AT&T, all handle this
correctly.

I just updated the tree, built on a Linux box, and all is happy. Corinna,
please let me know if you're still having problems, and if the command
above does not work for you. If it doesn't, I have no problem with 
adding a workaround something like:

2000-03-30  Mumit Khan  <khan AT xraylith DOT wisc DOT edu>

	* Makefile.in (subdirs): Workaround for broken shells.

Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/winsup/mingw/Makefile.in,v
retrieving revision 1.1.1.1
diff -u -3 -p -r1.1.1.1 Makefile.in
--- Makefile.in	2000/02/17 19:38:31	1.1.1.1
+++ Makefile.in	2000/03/31 03:43:55
@@ -212,7 +212,8 @@ install: all
 	@$(MAKE) subdirs DO=$@ $(FLAGS_TO_PASS)
 
 subdirs: force
-	@for i in $(SUBDIRS); do \
+	@test -z "$(SUBDIRS)" && exit 0; \
+	for i in $(SUBDIRS); do \
 	   echo "Making $(DO) in $${i}..." ; \
 	   if [ -d ./$$i ] ; then \
 	     if (rootme=`pwd`/ ; export rootme ; \


Regards,
Mumit


- Raw text -


  webmaster     delorie software   privacy  
  Copyright © 2019   by DJ Delorie     Updated Jul 2019