Mailing-List:	contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender:	cygwin-apps-owner AT cygwin DOT com
List-Subscribe:	<mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin-apps/>
List-Post:	<mailto:cygwin-apps AT cygwin DOT com>
List-Help:	<mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To:	mailing list cygwin-apps AT cygwin DOT com
Date:	Tue, 15 Jan 2002 12:58:26 +0100
From:	Corinna Vinschen <vinschen AT redhat DOT com>
To:	cygwin-apps AT cygwin DOT com
Subject:	Re: cygpath patch
Message-ID:	<20020115125826.W2015@cygbert.vinschen.de>
Reply-To:	cygwin-apps AT cygwin DOT com
Mail-Followup-To:	cygwin-apps AT cygwin DOT com
References:	<20020115021409 DOT 19064 DOT qmail AT web20001 DOT mail DOT yahoo DOT com>
Mime-Version:	1.0
In-Reply-To:	<20020115021409.19064.qmail@web20001.mail.yahoo.com>
User-Agent:	Mutt/1.3.22.1i

On Mon, Jan 14, 2002 at 06:14:09PM -0800, Joshua Franklin wrote:
> Well, I've attached the patch in case anyone is 
> interested. It changes cygpath.exe to include the
> options:
>   -A|--allusers         use `All Users' directories
> instead of current user
>   -D|--desktop          print `Desktop' directory
>   -P|--smprograms       print Start Menu `Programs'
> directory

Thanks for the patch.  I applied it to my local sandbox but I
was somewhat surprised about the behaviour since

  cygpath -D -A

has another result as

  cygpath -A -D

which is not what a user expects.  The options should be scanned
and after the scanning they should be evaluated (right after the
switch statement).  Would you mind to change your patch accordingly?

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin AT cygwin DOT com
Red Hat, Inc.

- Raw text -