autotest-inotify 0.0.5 → 0.0.6

data/README.rdoc

@@ -32,7 +32,7 @@ file:
 
 == THANK YOU:
 
-Inspiration for this gem came from work done by Alban Peignier's[1], and Sven
+Inspiration for this gem came from work done by Alban Peignier[1], and Sven
 Schwyn[2]
 
 1. http://blog.tryphon.org/alban/archives/2007/10/26/inotify-support-for-autotest/
@@ -42,7 +42,7 @@ Schwyn[2]
 
 (The MIT License)
 
-Copyright (c) 2009 Eric A. Wollesen
+Copyright (c) 2010 Eric A. Wollesen
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/lib/autotest/inotify.rb

@@ -6,7 +6,6 @@ require "rbconfig"
 require "rb-inotify"
 
 
-
 Autotest.add_hook :initialize do |at|
   include Autotest::Inotify
   at.setup_inotify_if_running_linux
@@ -45,13 +44,16 @@ module Autotest::Inotify
 
       def wait_for_changes
         @changed_files = {}
+        @waited = true
         hook :waiting
-        @notifier.process
+        @notifier.process while @changed_files.empty?
       end
 
       def find_files_to_test(files=nil)
-        if first_time_run?
-          select_all_tests
+        if not @waited
+          unless options[:no_full_after_start]
+            select_all_tests
+          end
         else
           p @changed_files if options[:verbose]
           hook :updated, @changed_files
@@ -62,19 +64,27 @@ module Autotest::Inotify
     end
   end
 
-  def first_time_run?
-    self.last_mtime.to_i.zero?
-  end
-
   def setup_inotify
     @notifier = INotify::Notifier.new
-    self.find_files.keys.each do |filename|
-      @notifier.watch(filename, :modify) do |event|
-        handle_file_event(event)
+    files = self.find_files.keys
+    dirs = files.map{|f| File.dirname( f )}.uniq
+    # Watch directories to catch delete/move swap patterns as well as direct
+    # modifications.  This handles, e.g. :w in vim.
+    dirs.each do |dir|
+      @notifier.watch(dir, :all_events) do |event|
+        if event_of_interest?(event.flags) &&
+            files.include?(event.absolute_name)
+          handle_file_event(event)
+        end
       end
     end
   end
 
+  def event_of_interest?(flags)
+    flags.include?(:modify) ||
+      flags.include?(:moved_to)
+  end
+
   def handle_file_event(event)
     @changed_files[event.absolute_name] = Time.now
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: autotest-inotify
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: false
   segments: 
   - 0
   - 0
-  - 5
-  version: 0.0.5
+  - 6
+  version: 0.0.6
 platform: ruby
 authors: 
 - Eric Wollesen
@@ -94,6 +94,6 @@ rubyforge_project:
 rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
-summary: Use libinotify (on Linux) instead of filesystem polling.
+summary: Teaches autotest to use libinotify (on Linux) instead of filesystem polling.
 test_files: []