autotest-inotify 0.0.4 → 0.0.5

data/README.rdoc

@@ -32,7 +32,7 @@ file:
 
 == THANK YOU:
 
-Inspiration for this gem came from work done by Alban Peignier[1], and Sven
+Inspiration for this gem came from work done by Alban Peignier's[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) 2010 Eric A. Wollesen
+Copyright (c) 2009 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

@@ -5,13 +5,16 @@ require "autotest"
 require "rbconfig"
 require "rb-inotify"
 
-Autotest.add_hook :initialize do
+
+
+Autotest.add_hook :initialize do |at|
   include Autotest::Inotify
-  setup_inotify_if_running_linux
+  at.setup_inotify_if_running_linux
 end
 
+
 ##
-# Autotest::Inofity
+# Autotest::Inotify
 #
 # == FEATURES:
 # * Use Linux's inotify instead of filesystem polling
@@ -21,14 +24,16 @@ end
 # require "autotest/inotify"
 module Autotest::Inotify
 
-  private
-
   def setup_inotify_if_running_linux
     if running_linux?
       override_autotest_methods
+      setup_inotify
     end
   end
 
+
+  private
+
   def running_linux?
     /linux/i === RbConfig::CONFIG["host_os"]
   end
@@ -41,17 +46,14 @@ module Autotest::Inotify
       def wait_for_changes
         @changed_files = {}
         hook :waiting
-        @notifier.process while @changed_files.empty?
+        @notifier.process
       end
 
       def find_files_to_test(files=nil)
         if first_time_run?
-          setup_inotify
-          unless options[:no_full_after_start]
-            select_all_tests
-          end
+          select_all_tests
         else
-          p @changed_files if $v
+          p @changed_files if options[:verbose]
           hook :updated, @changed_files
           select_tests_for_changed_files
         end
@@ -61,30 +63,18 @@ module Autotest::Inotify
   end
 
   def first_time_run?
-    @notifier.nil?
+    self.last_mtime.to_i.zero?
   end
 
   def setup_inotify
     @notifier = INotify::Notifier.new
-    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
+    self.find_files.keys.each do |filename|
+      @notifier.watch(filename, :modify) do |event|
+        handle_file_event(event)
       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: 23
+  hash: 21
   prerelease: false
   segments: 
   - 0
   - 0
-  - 4
-  version: 0.0.4
+  - 5
+  version: 0.0.5
 platform: ruby
 authors: 
 - Eric Wollesen
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-09-19 00:00:00 -06:00
+date: 2011-01-12 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -94,6 +94,6 @@ rubyforge_project:
 rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
-summary: Teaches autotest to use libinotify (on Linux) instead of filesystem polling.
+summary: Use libinotify (on Linux) instead of filesystem polling.
 test_files: []