Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stat the link itself instead of file it refers to #765

Merged
merged 1 commit into from Oct 25, 2014

Conversation

@shrijeet
Copy link

shrijeet commented Sep 4, 2014

If path is a symbolic link and target doesn't exist, we may incorrectly throw
file not found exception. This changes fixes the issue by using File.lstat to
stat the path.

If path is a symbolic link and target doesn't exist, we may incorrectly through
file not found exception. This changes fixes the issue by using File.lstat to
stat the path.
jordansissel added a commit that referenced this pull request Oct 25, 2014
Stat the link itself instead of file it refers to
@jordansissel jordansissel merged commit 310c39f into jordansissel:master Oct 25, 2014
@jordansissel

This comment has been minimized.

Copy link
Owner

jordansissel commented Oct 25, 2014

<3

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 18, 2014
Stat the link itself instead of file it refers to
prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 27, 2014
Stat the link itself instead of file it refers to
jordansissel added a commit that referenced this pull request Apr 24, 2015
Stat the link itself instead of file it refers to
jordansissel added a commit that referenced this pull request Jun 20, 2016
Stat the link itself instead of file it refers to
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.