mixlib-shellout 1.6.0.rc.0 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 021c7f8efea2d6d11f349b4a1728d3039567b191
-  data.tar.gz: b9d6684c7b751462d6e4c2cea479f39474e3ee6d
+  metadata.gz: 1c5438feb4cca570fca5c9a9e1566ae6df1696d7
+  data.tar.gz: 42219e3a8e1a6300d72aeb63924dab765bbc5f0b
 SHA512:
-  metadata.gz: e4a1e30bb080f4bad1697480360acc76b852a32d46d1f8e350e2ce1a0c9024734a5f3ce0377c56ad5a72207084410bcb32eaa1477bb25e7e1ea51f4a1e840e24
-  data.tar.gz: 674076ca9e875c2b2b55d604f6ce3279c99e87014002429e51e1d1eeaac245947dbbbddd3811d32df5ca17e97910687ffaed91f9de7e41950b961be7cfcc27dd
+  metadata.gz: 31a81beb7022501bff7f57ff17eb91f97fc472f1b86199260d06910b8d1a05d68ac04d4908da39b7ecf5b8170f79402f0702a21d06ac044f4ae11b340a7545f6
+  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: ruby
 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
@@ -51,15 +51,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