Skip to content

Fix resolving absolute symlinks (on at least Win32)#4169

Closed
csware wants to merge 1 commit into
libgit2:masterfrom
csware:absolute-symlink
Closed

Fix resolving absolute symlinks (on at least Win32)#4169
csware wants to merge 1 commit into
libgit2:masterfrom
csware:absolute-symlink

Conversation

@csware
Copy link
Copy Markdown
Contributor

@csware csware commented Mar 20, 2017

The symlink destination is always concatenated to the original path.

See issue #4168.

based on the implementation I assume that this is also broken for *nix symlinks.

The symlink destination is always concatenated to the original path.

See issue #4168.

Signed-off-by: Sven Strickroth2 <[email protected]>
pks-t added a commit that referenced this pull request Mar 20, 2017
@pks-t
Copy link
Copy Markdown
Member

pks-t commented Mar 20, 2017

I've manually merged this via 3105992. I've changed your commit to use git_buf_sets instead of git_buf_clear/git_buf_puts and added another test which verifies your fix.

Thanks for this fix! 😄

@pks-t pks-t closed this Mar 20, 2017
@csware csware deleted the absolute-symlink branch March 20, 2017 15:50
yuyichao added a commit to JuliaLang/julia that referenced this pull request Apr 29, 2017
Ref libgit2/libgit2#4169

* Test this on unix
* Do not write to user's global gitconfig
yuyichao added a commit to JuliaLang/julia that referenced this pull request Apr 29, 2017
Ref libgit2/libgit2#4169

* Test this on unix
* Do not write to user's global gitconfig
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants