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
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
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.