process_starter 1.0.0 → 1.0.1

data/Rakefile

@@ -0,0 +1,8 @@
+require 'rake/testtask'
+
+Rake::TestTask.new do |t|
+  t.libs << 'test'
+end
+
+desc "Run tests"
+task :default => :test

data/lib/process_starter.rb

@@ -1,14 +1,20 @@
 # 2012 (c) LineAct / www.lineact.com
 
+require 'rubygems'
+require 'fcntl'
+
+# https://github.com/rtomayko/posix-spawn
+begin; require( 'posix/spawn' ) ; rescue LoadError ; end
+
 module ProcessStarter
-  begin require 'posix/spawn' rescue LoadError; end
 
   WINRUN = (RUBY_PLATFORM =~ /mswin|mingw/) ? true : false # run in windows?
-  HAVE_SPAWN = defined?(spawn)                     # have Ruby 1.9 spawn method
-  HAVE_POSIX_SPAWN = defined?(POSIX::Spawn::spawn) # loaded posix/spawn lib
+  HAVE_SPAWN = defined?(spawn) ? true : false                     # have Ruby 1.9 spawn method
+  HAVE_POSIX_SPAWN = defined?(POSIX::Spawn::spawn) ? true : false # loaded posix/spawn lib
+  HAVE_PSEXEC = WINRUN ? ((`psexec /?` rescue "") =~ /computer/ ? true : false) : false
 
-  def log(m)
-    defined?(RAILS_DEFAULT_LOGGER) ? RAILS_DEFAULT_LOGGER.info(m) : puts m
+  def self.log(m)
+    defined?(RAILS_DEFAULT_LOGGER) ? RAILS_DEFAULT_LOGGER.info(m) : puts(m)
   end
 
   # Starts the process
@@ -16,40 +22,39 @@ module ProcessStarter
   # not inheritting file descriptors, so it finds and closes all opened handels 
   # works in windows and linux
   def self.start(s)
-    log.info "**************** INVOKING EXTERNAL COMMAND #{s}, WINRUN IS #{WINRUN}, HAVE_POSIX_SPAWN IS #{HAVE_POSIX_SPAWN}, HAVE_SPAWN is #{HAVE_SPAWN}"
+    log "**************** INVOKING EXTERNAL COMMAND #{s}, WINRUN IS #{WINRUN}, HAVE_PSEXEC IS #{HAVE_PSEXEC}, HAVE_POSIX_SPAWN IS #{HAVE_POSIX_SPAWN}, HAVE_SPAWN is #{HAVE_SPAWN}"
     if WINRUN
-      if system("psexec -d #{s}")
-        log.info "Executed using PSEXEC"
-      elsif system("start cmd /c #{s}")
-        log.info "Executed using START cmd /c (inherits descriptors (?) - bad)"
+      if HAVE_PSEXEC
+        log "starting using psexec"
+        r = system("psexec -d #{s}")
       else
-        log.info "Execute failed!"
+        log "starting using START cmd /c (inherits descriptors (?) - bad)"
+        system("start cmd /c #{s}")
       end
     else
-      log.info "Executing in Linux style"
+      log "Executing in Linux style"
       if HAVE_POSIX_SPAWN
         close_them = Hash[ close_io().map {|x| [x, :close]} ]
-        log.info "POSIX::Spawn::spawn with close descriptors: #{close_them.inspect}"
-        pid  = POSIX::Spawn::spawn( s, close_them)
+        log "POSIX::Spawn::spawn with close descriptors: #{close_them.inspect}"
+        Process.detach POSIX::Spawn::spawn( s, close_them) 
       elsif HAVE_SPAWN
         # we hope on http://www.ruby-doc.org/core-1.9.3/Process.html#method-c-spawn
         # [close_others => true] option
         # TODO: check it
-        log.info "Ruby 1.9 spawn"
-        spawn( s )
+        log "Ruby 1.9 spawn"
+        Process.detach spawn( s ) 
       else
-        log.info "fork{ close_io; exec }"
-        Process.detach fork{
+        log "fork{ close_io; exec }"
+        Process.detach fork {
           close_io(true)
           exec(s)
-        }
+        } 
       end
-
     end
   end
 
   # based on close_io from: http://daemons.rubyforge.org/Daemonize.html
-  def self.close_io( do_close=false )
+  def self.close_io( do_close=false, debug = false )
     close_them = []
     # Make sure all input/output streams are closed
     # Part I: close all IO objects (except for STDIN/STDOUT/STDERR)
@@ -72,12 +77,15 @@ module ProcessStarter
     # Part II: close all file decriptors (except for STDIN/STDOUT/STDERR)
     ios = Array.new(8192) {|i| IO.for_fd(i) rescue nil}.compact
     ios.each do |io|
-  #    puts "qqq"
-  #    puts io.to_i
+      puts "qqq" if debug
+      puts io.to_i if debug
       next if io.fileno < 3
-  #    puts "checking flag"
-  #    puts io.fcntl(Fcntl::F_GETFD) & Fcntl::FD_CLOEXEC
-      next if (io.fcntl(Fcntl::F_GETFD) & Fcntl::FD_CLOEXEC != 0)
+      puts "checking flag" if debug
+      r = begin; io.fcntl(Fcntl::F_GETFD) & Fcntl::FD_CLOEXEC != 0 rescue false; end
+      puts r
+#      puts io.fcntl(Fcntl::F_GETFD) & Fcntl::FD_CLOEXEC
+#      next if (io.fcntl(Fcntl::F_GETFD) & Fcntl::FD_CLOEXEC != 0)
+      next if r
   #    puts "adding"
       close_them << io.to_i
       io.close if do_close
@@ -87,4 +95,8 @@ module ProcessStarter
     close_them.uniq
   end
 
-end
+end
+
+# f = File.open("test2.txt","w")
+# d = ProcessStarter.close_io()
+# puts "Parent descriptors: #{d.inspect}"

data/test/test2child.arb

@@ -0,0 +1,10 @@
+puts "im a child"
+#require File.join( File.expand_path(File.dirname(__FILE__)), "..","lib","process_starter.rb" )
+#d = ProcessStarter.close_io()
+#puts "im a child and my descriptors: #{d.inspect}"
+#puts "im a child and my descriptors:"
+
+puts `ls -l /proc/#{Process.pid}/fd` rescue nil
+`echo 123 >#{File.expand_path(File.dirname(__FILE__))}/chout2`
+sleep ARGV[0].to_i rescue 0
+puts "i am child and i finish"

data/test/test_process_starter.rb

@@ -0,0 +1,47 @@
+require 'test/unit'
+require 'process_starter'
+
+class ProcessStarterTest < Test::Unit::TestCase
+#  def test_english_hello
+#    assert_equal "hello world",
+#      Hola.hi("english")
+#  end
+
+#  def test_visually_one
+#    puts "I am test_visually_one"
+#  end
+
+  def test_visually
+    puts "I am test_visually"
+    f = File.open("test2.txt","w")
+    puts "I opened test2.txt, it's fd=#{f.to_i}"
+    fb = File.open("test2b.txt","w" )
+    #fb.fcntl(Fcntl::F_SETFD,fb.fcntl(Fcntl::F_GETFD) | Fcntl::FD_CLOEXEC )
+    puts "I opened test2b.txt, it's fd=#{fb.to_i}"
+
+    require 'socket'      # Sockets are in standard library
+    host = '0.0.0.0'
+    port = 2008
+    s = TCPServer.open( port )
+    puts "I opened socket on port 2006, it's fd=#{s.to_i}"
+
+#    d = ProcessStarter.close_io()
+    puts "I am parent, my descriptors:"
+    puts ProcessStarter::WINRUN ? "non avail" : `ls -l /proc/#{Process.pid}/fd`
+
+#    r = ProcessStarter.start("ruby test/test2child.arb 120 >test/chout 2>&1")
+    
+    ProcessStarter.start("ruby test/test2child.arb 120")
+    #stat = Process::waitpid(r)
+    
+    #puts "i started child, r=#{r}, status = #{stat.inspect}"
+    if ProcessStarter::WINRUN
+      puts "We are in Windows, so i just sleep for 2 minutes so you can check handles manually, for ex using ProcessExplorer"
+      sleep 120
+    else
+      sleep 5
+    end
+     
+  end
+
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: process_starter
 version: !ruby/object:Gem::Version 
-  hash: 23
+  hash: 21
   prerelease: 
   segments: 
   - 1
   - 0
-  - 0
-  version: 1.0.0
+  - 1
+  version: 1.0.1
 platform: ruby
 authors: 
 - Pavel Vasev
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-09-21 00:00:00 +06:00
+date: 2012-09-23 00:00:00 +06:00
 default_executable: 
 dependencies: []
 
@@ -29,6 +29,9 @@ extra_rdoc_files: []
 
 files: 
 - lib/process_starter.rb
+- Rakefile
+- test/test_process_starter.rb
+- test/test2child.arb
 has_rdoc: true
 homepage: http://rubygems.org/gems/process_starter
 licenses: []
@@ -62,6 +65,6 @@ rubyforge_project:
 rubygems_version: 1.5.2
 signing_key: 
 specification_version: 3
-summary: Hola!
+summary: Process Starter / not inherit file descriptors
 test_files: []