D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 6068 - std.path has some issues for Windows user
Summary: std.path has some issues for Windows user
Status: RESOLVED WONTFIX
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: Other Windows
: P2 normal
Assignee: No Owner
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-28 05:21 UTC by Denis Shelomovskii
Modified: 2015-06-09 05:10 UTC (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Denis Shelomovskii 2011-05-28 05:21:26 UTC
getExt, getName and join doesn't support Windows's altsep, look at pull for D2's phobos:
https://github.com/D-Programming-Language/phobos/pull/63

Confusing (for me) behaviour of functions:

1. isabs - from
http://msdn.microsoft.com/en-us/library/aa365247%28v=vs.85%29.aspx
: "A single backslash, for example, "\directory" or "\file.txt". This is also referred to as an absolute path." and in .NET Framework Path.IsPathRooted returns true for such case, e.g. But I agree with phobos's way, and this behaviour is clearly written in documentation. Have it your way. But it affects next function.

2. join - if second path is "drive relative rooted":
assert( join(`a`, `\b`) == `\b` ); //good (like .NET Framework)
assert( join(`c:a`, `\b`) == `c:a\b` ); //why not `c:\b`?
assert( join(`c:\a`, `\b`) == `c:\a\b` ); //why not `c:\b`?

.NET Framework's Path.Combine for all cases shows `\b` (because `\b` is rooted for it).

Possible solutions:
1. We don't want to know about Microsoft's stupid "drive relative rooted" paths, let it be so:
assert( join(`a`, `\b`) == `a\b` );
assert( join(`c:a`, `\b`) == `c:a\b` );
assert( join(`c:\a`, `\b`) == `c:\a\b` );

2. We know about it:
assert( join(`a`, `\b`) == `\b` );
assert( join(`c:a`, `\b`) == `c:\b` );
assert( join(`c:\a`, `\b`) == `c:\b` );
Comment 1 Jonathan M Davis 2011-05-28 15:07:25 UTC
I would point out that Lars has revamped std.path, and his changes are likely to be up for review sometime soon (I believe that dsimcha's TempAlloc stuff is going to be reviewed first). And that's likely to change the behavior of some of std.path's stuff. Your issues here may be totally fixed in the code that Lars has done. So, I'm reluctant to spend time trying to fix what's currently there. As long as the pull request is good, it'll probably get merged in, but fixing the other issues should probably be left to Lars' changes, since I fully expect some version of that to get through the review process. We should keep these issues in mind however when Lars' changes are reviewed so that new code handles them in whatever the best way is.
Comment 2 Lars T. Kyllingstad 2012-02-01 14:31:36 UTC
Marking this as "won't fix", since the functions in question will soon be removed from Phobos.