-
Notifications
You must be signed in to change notification settings - Fork 248
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
Polling and double changes in 0.5.0 under OS X 10.8 #62
Comments
I have exactly this issue on mountain lion, but reverting to 0.4.7 didn't help. |
Same thing. Reverting to 0.4.7 fixed the issue. Waiting for the fix... |
I just pushed a commit to the master branch that would hopefully fix the constant use of polling on Mac. Could you guys try it out and tell me if the fix works for you? As for double change issue, I couldn't reproduce it. Are you seeing this issue while using Guard or do you use Listen in your own project? if so, could someone post a repeatable way to reproduce that issue? |
Rock on, master seems to fix the polling issue for me. 👍 FWIW, I also have not been able to repro the double change issue. |
+1 |
The changes on master fixed the polling issues over here! |
Splendid, that fixes it here as well. Thank you! Now that that works, I can confirm that double change appears to be a polling issue. I use Listen for my own project, without Guard. |
@Maher4Ever I tried https://gist.github.com/3703306 with new changes on master. I can confirm I still get the double change firing. let me know if there's any other details I can report (or tests I can run) to help narrow this down. |
I wasn't able to reproduce the double change firing with your gist on: OS X: 10.8.1 |
@thibaudgg Is there any sort of logging I can do to help figure out whats causing this double change fire when its in polling mode? |
@meeech sadly not yet. |
@Maher4Ever @thibaudgg some more details to share - maybe this will help track it down or reproduce Is it possible in polling there's something cheking file mtime AND some sort of hash of the file itself to see if it changed? so when just the mtime has changed, the event fires 1 time, but when the actual contents of file has changed, we end up with the event firing 2x |
I was also having polling and double issues and adding |
@meeech do you also use the master branch of Listen? |
yes I was. Sent from my iPhone On 2012-09-18, at 10:31 AM, Thibaud Guillaume-Gentil notifications@github.com wrote:
|
@meeech Are you experiencing the double change issue when making an edit a file via a particular editor (ex. vim or sublime) or even when running something like As for your suggestion to help fix the bug, you might be test that theory by adding some |
Since I am encountering the same issue as @meeech, here are a few more details from me. Using his gist, The issue only occurs when modifying a file. Editing and saving a file outputs two changes. Saving again without further modification results in one change. Making any other edits (including undo) and saving results in two changes. Effectively:
Tested with TextMate, Sublime Text, TextWrangler and TextEdit -- the editor does not seem to matter. |
I see the behavior @meeech and @brief described in 0.5.0 whenever it uses polling, but not when it doesn't have to fall back to polling. In 0.5.1/HEAD, it's the same. With the fsevent issue fixed, I don't get the double-run error unless I start with I'm on OSX 10.6.8, and tested with both Ruby 1.8.7 and 1.9.3. |
@Maher4Ever @brief gave a much better summary of the issue. |
@brief Thanks for the details, I was able to reproduce this issue. I'll try to create a spec to catch it. |
I think I have found the issue, I'll push a fix this week. |
@thibaudgg great! thanks |
This should have been fixed in the polling/double branch. Please can you give it a try by adding that in your Gemfile: gem 'listen', github: 'guard/listen', branch: 'polling/double' Thanks! |
@thibaudgg we have a winner! all good now over here - osx 10.6.8/ruby 1.8.7 |
@meeech Nice, thanks for the feedback. 0.5.2 released! |
Listen 0.5.0 breaks a few things for me under Mountain Lion:
Reverting to Listen 0.4.7 fixes these issues on my machine.
The text was updated successfully, but these errors were encountered: