Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Message-ID: <3D7FB4E3.35AF1C1E@pajhome.org.uk>
Date: Wed, 11 Sep 2002 22:25:55 +0100
From: Paul Johnston <paj AT pajhome DOT org DOT uk>
X-Accept-Language: en
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: Beginnings of a patch: /etc/hosts
References: <Pine DOT GSO DOT 4 DOT 44 DOT 0209111439280 DOT 1269-100000 AT slinky DOT cs DOT nyu DOT edu>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Hi,

Brilliant to see so much progress on this!

Are you sure that "CYGWIN_9*)" correctly catches Windows ME?

I think we're safe with the case issue, given that cygpath -W seems to return
the correct case.
However, I think adding this belt-and-braces check might be a good idea:

if [ ! -d "$WINETC" ]
then
  echo "ERROR: Directory $WINETC not found" >&2
  exit 2
fi

Apart from that, seems just about ready to go. Not sure if the change history
is necessary though, at least not until it's first checked-in.

Paul


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/