This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] cygpath.cc


Hi Corinna,

>AFAICS, the patch is ok.

Fine.

>Just two question:
>
>- The -s and -l options are only valid with the -w option.  Shouldn't
>  either the usage reflect that or the -s and -l options imply -w
>  automatically?  It's not *that* obvious for the user that s/he
>  has to use `cygpath -w -l ...'.

I always had in mind that it would be great to implement the options once
for -u, too.
Example:

$ cygdrive -u `cygdrive -asw \`cygdrive -u 'C:\Dokumente und
Einstellungen\All Users\.bashrc'\``
/cygdrive/c/DOKUME~1/ALLUSE~1/BASHRC~2

It would be nice to have instead:

$ cygdrive -us 'C:\Dokumente und Einstellungen\All Users\.bashrc'
/cygdrive/c/DOKUME~1/ALLUSE~1/BASHRC~2

Another example:
At least on my box W2K is converting TEMP automatically into short form if I
open a console. This is quite inconvenient in bash and I would like to have
rather the long form:

$ echo $TEMP
/cygdrive/c/DOKUME~1/joehni/LOKALE~1/Temp

Prefered call:
$ export TEMP=`cygpath -ul $TEMP`
$ echo $TEMP
/cygdrive/c/Dokumente\ und\ Einstellungen/joehni/Lokale\ Einstellungen/Temp

In any case, the normal usage of cygpath should not care about -s or -l,
since you would normally only convert a Unix path into a DOSish notation
(which is by default the long form anyway). Both options are IMHO for very
special purposes.

What bothers me more, is that you cannot convert a short path into the long
version when check_case:strict is set, since the conversion functions in the
cygwin1.dll will reject that path as non-matching. But I am not really sure,
whether this functions should look for the alternative physical name or not.
Another problem arises with Java CLASSPATHs, since their element may be also
files instead of directories, which prevents a conversion unfortunately,
too.

Regards
Jörg


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]