mixlib-shellout 1.6.0.rc.0-x86-mingw32 → 1.6.1-x86-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 090582646d44022c8d0f06d29b2a6bea06875aad
-  data.tar.gz: b9d6684c7b751462d6e4c2cea479f39474e3ee6d
+  metadata.gz: b6d2f3bf553099c4188b33584f3030068a2e46e4
+  data.tar.gz: 42219e3a8e1a6300d72aeb63924dab765bbc5f0b
 SHA512:
-  metadata.gz: 54b79dc8fa8a509cadf3e6738931b586819ba254d9d76c23cf8bad4492dde20dbca7975f57f91e7fe5f68c600d81640564044fd1cc1df3d6c8837857fa593c80
-  data.tar.gz: 674076ca9e875c2b2b55d604f6ce3279c99e87014002429e51e1d1eeaac245947dbbbddd3811d32df5ca17e97910687ffaed91f9de7e41950b961be7cfcc27dd
+  metadata.gz: d622c037ca945587e3e14bcfe8109b91b5dd1f797e3afd9b1070e65d325462f71e40001b0ea1adcb7b3bbddd40249b9df29bb4e91a2e1e47a34e456257afde4c
+  data.tar.gz: 393c9b543d50e6c03f00b478a4903ea42cb4edb9fb9e93a9eabfbef9e2a541a837403ea77dff1d6dda8808dd64a1e0a5ce71545e551ed158b17d4fe85dedde91

data/lib/mixlib/shellout/unix.rb

@@ -121,8 +121,8 @@ module Mixlib
 
       def set_user
         if user
-          Process.euid = uid
           Process.uid = uid
+          Process.euid = uid
         end
       end
 
@@ -211,31 +211,6 @@ module Mixlib
         STDIN.sync = true if input
       end
 
-      # When a new process is started with chef, it shares the file
-      # descriptors of the parent. We clean the file descriptors
-      # coming from the parent to prevent unintended locking if parent
-      # is killed.
-      # NOTE: After some discussions we've decided to iterate on file
-      # descriptors upto 256. We believe this  is a reasonable upper
-      # limit in a chef environment. If we have issues in the future this
-      # number could be made to be configurable or updated based on
-      # the ulimit based on platform.
-      def clean_parent_file_descriptors
-        # Don't clean $stdin, $stdout, $stderr, process_status_pipe.
-        3.upto(256) do |n|
-          # We are checking the fd for error pipe before attempting to
-          # create a file because error pipe will auto close when we
-          # try to create a file since it's set to CLOEXEC.
-          if n != @process_status_pipe.last.to_i
-            begin
-              fd = File.for_fd(n)
-              fd.close if fd
-            rescue
-            end
-          end
-        end
-      end
-
       def configure_parent_process_file_descriptors
         # Close the sides of the pipes we don't care about
         stdin_pipe.first.close
@@ -309,8 +284,6 @@ module Mixlib
 
           configure_subprocess_file_descriptors
 
-          clean_parent_file_descriptors
-
           set_group
           set_user
           set_environment
@@ -318,7 +291,7 @@ module Mixlib
           set_cwd
 
           begin
-            command.kind_of?(Array) ? exec(*command) : exec(command)
+            command.kind_of?(Array) ? exec(*command, :close_others=>true) : exec(command, :close_others=>true)
 
             raise 'forty-two' # Should never get here
           rescue Exception => e

data/lib/mixlib/shellout/version.rb

@@ -1,5 +1,5 @@
 module Mixlib
   class ShellOut
-    VERSION = "1.6.0.rc.0"
+    VERSION = "1.6.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mixlib-shellout
 version: !ruby/object:Gem::Version
-  version: 1.6.0.rc.0
+  version: 1.6.1
 platform: x86-mingw32
 authors:
 - Opscode
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-08 00:00:00.000000000 Z
+date: 2014-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -79,15 +79,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.4
 signing_key: 
 specification_version: 4
 summary: Run external commands on Unix or Windows