process_starter 1.0.10 → 1.0.11

data/README

@@ -15,12 +15,12 @@ ProcessStarter.allowed_methods = [:psexec,:start,:spawn,:fork]
 specify which methods are allowed to start the process. So we have psexec 
 (if found on the target machine) and 'cmd /c start ...' for windows,
 and spawn (if in ruby 1.9) or fork in linux. This is the default,
-but you can redefine it, for example
+but you can redefine it, for example:
 ProcessStarter.allowed_methods = [:start,:fork]
 
 Also you can set allowed methods in command call:
 
-ProcessStarter.start("somecmd arg1 arg2",[:start,:fork])
+ProcessStarter.start("somecmd arg1 arg2", [:start,:fork] )
 
 This overrides ProcessStarter.allowed_methods values.
 

data/lib/process_starter.rb

@@ -30,8 +30,12 @@ module ProcessStarter
   # works in windows and linux
   def self.start(s, allowed_methods = nil )
     allowed_methods ||= self.allowed_methods
+
     log "********* ProcessStarter.start INVOKING EXTERNAL COMMAND #{s}, WINRUN IS #{WINRUN}, HAVE_POSIX_SPAWN IS #{HAVE_POSIX_SPAWN}, HAVE_SPAWN is #{HAVE_SPAWN}, allowed_methods are #{allowed_methods.inspect}"
     if WINRUN
+#      if s.index("'")
+#        s = s.gsub("'","\\'")
+#      end    
       @@have_psexec ||= allowed_methods.include?(:psexec) ? ((`psexec /?` rescue "") =~ /computer/ ? 2 : 1) : -1
       if @@have_psexec == 2 and allowed_methods.include?(:psexec)
         log "starting using psexec"
@@ -40,10 +44,29 @@ module ProcessStarter
         r = system("psexec -d -w #{Dir.pwd} #{s}")
       else
         log "starting using START cmd /c (inherits descriptors (?) - bad)"
-        system("start cmd /c #{s}")
+        close_io(true)
+        system("start /b cmd /c #{s}")
       end
     else
       log "Executing in Linux style"
+
+      # fix "s" for stdin, stdout, stderr
+      nilport = WINRUN ? "NUL" : "nil"
+      # not specified stdin?
+      if not s =~ /</
+        s = s + " <#{nilport}"
+      end
+      # not specified both stdout and stderr?
+      if not s =~ />/
+        s = s + " >#{nilport} 2>#{nilport}"
+      end
+      # not specified stderr?
+      if not s =~ /2>/
+        s = s + " 2>#{nilport}"
+      end
+      # we omit the case: "program 2>nil", which specifies that stdout is not redirected, which means that 
+      # if user specify 2>, he is clever, and has to specify 1> too.
+
       if HAVE_POSIX_SPAWN and allowed_methods.include?(:posix_spawn)
         close_them = Hash[ close_io().map {|x| [x, :close]} ]
         log "POSIX::Spawn::spawn with close descriptors: #{close_them.inspect}"

data/test/test2child.arb

@@ -1,4 +1,5 @@
 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}"

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: process_starter
 version: !ruby/object:Gem::Version 
-  hash: 3
+  hash: 1
   prerelease: 
   segments: 
   - 1
   - 0
-  - 10
-  version: 1.0.10
+  - 11
+  version: 1.0.11
 platform: ruby
 authors: 
 - Pavel Vasev
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-10-18 00:00:00 Z
+date: 2012-02-22 00:00:00 Z
 dependencies: []
 
 description: A gem for Ruby 1.8 to start process in parallel, 1) don't wait for it finish 2) dont inherit io file handles 3) both in windows/linux. Ruby 1.9 dont need it because it has spawn method.