Looking at the patch, it
appears to be fairly
"kludgy". I'd be much
happier to see someone
develop a patch that
builds on Tcl's ability to
support virtual filesystems
(available for years now,
ever since Tcl 8.4.0), so
that the weird Cygwin
filesystem can be made
available to Tcl on its
own terms, not as a collection
of tweaks and hacks #ifdef'd
into the native filesystem
support.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
You'll need to submit a new Patch issue, as SF only permits
the submitter (and project techs/admins) to attach or remove
files. If you're not logged in at submission, that's just
too bad. And we (the Tcl people) can't fix that; it's a
general SF policy.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I don't think it's possible to do this with
the virtual filesystem API, as the UNIX
path functions handle '/' separately. To
me this patch looks like the way to go.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Cygwin network pathname support
Logged In: YES
user_id=80530
What version of Tcl
is the patch for?
Please indicate in
the "Group" form.
Logged In: YES
user_id=80530
Looking at the patch, it
appears to be fairly
"kludgy". I'd be much
happier to see someone
develop a patch that
builds on Tcl's ability to
support virtual filesystems
(available for years now,
ever since Tcl 8.4.0), so
that the weird Cygwin
filesystem can be made
available to Tcl on its
own terms, not as a collection
of tweaks and hacks #ifdef'd
into the native filesystem
support.
Logged In: YES
user_id=1336038
I updated it for current CVS, we use it with 8.4.1. It wa
probably forward proted from an earlier version.
(The group form appears to be accessible only for poroject
admins.)
Logged In: YES
user_id=79902
You'll need to submit a new Patch issue, as SF only permits
the submitter (and project techs/admins) to attach or remove
files. If you're not logged in at submission, that's just
too bad. And we (the Tcl people) can't fix that; it's a
general SF policy.
Committed to bug-1536227 branch
I don't think it's possible to do this with
the virtual filesystem API, as the UNIX
path functions handle '/' separately. To
me this patch looks like the way to go.
Fixed in core-8-5-branch and trunk
It turns out that the current cygwin
implementation is not quite correct:
On Windows:
% file split //a/b/c/d
//a/b c d
On Cygwin:
% file split //a/b/c/d
// a b c d
There are various tickets explaining
why the win32 answer is correct, the
Cygwin implementation should do the same thing.
B.T.W. The QNX implementation is not
correct either:
% file split //12b/c
//12/b/c
And let's see if I can address some of dgp's points below.
Cygwin implementation now matches the Windows
implementation, as indicated below.
Fixed in core-8-5-branch and trunk.