-
Notifications
You must be signed in to change notification settings - Fork 626
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
Vendor the "right" fsnotify #10
Conversation
Fixes #7 |
@freeformz IIUC, this should eliminate the dependency on |
This is the version that hpcloud/tail vendors
The version of fsnotify that is vendored doesn't use it.
@Random-Liu done. |
@freeformz Quick question. I'm not quite familiar with Godeps. Could you explain the last 2 commits a bit? The first 2 LGTM. |
All right, understood the 3rd one. https://github.com/tools/godep/blob/master/Changelog.md |
Before v71 we didn't preserve permissions of the files copied in, now we do. This is particularly important for executable files. The last one doesn't really matter that much, it looks like a more exact tag was added at some point and we're now picking it up. |
@freeformz Thanks a lot! LGTM! |
This is the version that hpcloud/tail vendors