https://issues.dlang.org/show_bug.cgi?id=15776

Timothee Cour <timothee.cour2@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |timothee.cour2@gmail.com

--- Comment #1 from Timothee Cour <timothee.cour2@gmail.com> ---
(In reply to Timothee Cour from comment #0)
> doc says, for symlink(original, link):
> 
> Relative paths are relative to the current working directory, not the files being linked to or from.
> 
> it should be:
> link is relative to cwd (if relative), original is relative to link (if
> relative), so that in the example below:
> 
> buildPath(link.dirName, original).buildNormalizedPath
> =buildPath(`d01/pz2.txt`.dirName, `../z2.txt`).buildNormalizedPath
> =`z2.txt` = actual location of original wrt cwd, which is != original.
> (and this is still valid for directory symlinks)
> Perhaps renaming original to original_wrt_link would make this more obvious
> 
> 
> tree
> .
> ├── d01
> └── z2.txt
> 
> std.file.symlink(`../z2.txt`, `d01/pz2.txt`)
> 
> .
> ├── d01
> │   └── pz2.txt -> ../z2.txt
> └── z2.txt


Also, docs should specify precisely what happens when link ends with a '/'

--

https://issues.dlang.org/show_bug.cgi?id=15776

ag0aep6g@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ag0aep6g@gmail.com
           Hardware|x86                         |All
                 OS|Mac OS X                    |All

--- Comment #2 from ag0aep6g@gmail.com ---
(In reply to Timothee Cour from comment #0)
> doc says, for symlink(original, link):
> 
> Relative paths are relative to the current working directory, not the files being linked to or from.
> 
> it should be:
> link is relative to cwd (if relative), original is relative to link (if
> relative), so that in the example below:

Already addressed by <https://github.com/D-Programming-Language/phobos/pull/3966>. See <http://dlang.org/phobos-prerelease/std_file.html#.symlink> for the new documentation.

(In reply to Timothee Cour from comment #1)
> Also, docs should specify precisely what happens when link ends with a '/'

This is still undocumented.

--

https://issues.dlang.org/show_bug.cgi?id=15776

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P4

--

https://issues.dlang.org/show_bug.cgi?id=15776

--- Comment #3 from dlangBugzillaToGithub <robert.schadek@posteo.de> ---
THIS ISSUE HAS BEEN MOVED TO GITHUB

https://github.com/dlang/phobos/issues/10165

DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB

--