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

added test for links #925

Closed
wants to merge 1 commit into from
Closed

added test for links #925

wants to merge 1 commit into from

Conversation

@jtangas
Copy link

@jtangas jtangas commented Jul 7, 2013

This allows fabric to test if a file exists via symlink since a standard exists will not return the proper result.

Makes useful addition to test if a remote directory is a symlink or not, helps expand on "exists" method to give a broader scope for checking files and paths.

The "e" flag does not apply to links soft or hard.

This allows fabric to test if a file exists via symlink since a standard exists will now return the proper result.
bitprophet added a commit that referenced this pull request Jul 7, 2013
bitprophet added a commit that referenced this pull request Jul 7, 2013
bitprophet added a commit that referenced this pull request Jul 7, 2013
bitprophet added a commit that referenced this pull request Jul 7, 2013
bitprophet added a commit that referenced this pull request Jul 7, 2013
@bitprophet
Copy link
Member

@bitprophet bitprophet commented Jul 7, 2013

Thanks! I'm a moron and this should've been on master after all; thankfully it's trivial to go from stable => dev, so I simply merged master into my integration branch :)

Also made some tweaks & added tests + changelog (vs asking you to to so) since I felt bad for punting your tickets around ;)

@bitprophet
Copy link
Member

@bitprophet bitprophet commented Jul 7, 2013

As usual...Github didn't notice. I really hate Pull Requests sometimes. Closing manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants