posix-spawn 0.3.12 → 0.3.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5e7aaeb2727c0c2d0d4b1055fee451289ddd510b
-  data.tar.gz: 87bcf9d8cc5cd759a6dc985a683b2a6d44dd8152
+SHA256:
+  metadata.gz: e80888255d57122ba2ad477c456582e0ebfdd05a61b175c60577ee8b38e579ad
+  data.tar.gz: 12d37e71310f0127b75b0f31da196c9c7554cac29a31d63754583f7737132190
 SHA512:
-  metadata.gz: 4f94bf7abd676780045ffb4c7d010cb0cc626d1fe6bbe84fab3126c4764a14f3828266db52f5ffa1eed5827ce0cac988b53f443bf9fed2bfd37110f6b55092f5
-  data.tar.gz: 20f06443465c1b96e06f988ca0c9b74df329cb13bbc0cb132217fa9453d35680e539de59492bfb2fe65bdd37c5c3d2b36a970899fd5fe860a04c0a9206530a09
+  metadata.gz: 3371e197b9b63ec2dce0b64d070bce8628ccb63112bcd14e8f7426891d1acf4d4ce4bc9558f44a832eaabd194b4b11a7feb2f12db2dd1d6db60968b8b9bc1a2a
+  data.tar.gz: 4d63126f9866a66ac6cc50d811be076939e1048f490aa40c55db1931398e016d225c3ae3da939a93646be551e6b207066f1dd4d2a1ee4f7dfe859a4404325242

data/.travis.yml

@@ -0,0 +1,9 @@
+sudo: false
+language: ruby
+rvm:
+  - 1.9
+  - 2.0
+  - 2.2
+  - 2.4
+  - 2.6
+  - ruby-head

data/ext/posix-spawn.c

@@ -45,7 +45,7 @@ static VALUE rb_mPOSIXSpawn;
  *       an actual fd number:
  *         - The symbols :in, :out, or :err for fds 0, 1, or 2.
  *         - An IO object. (IO#fileno is returned)
- *         - A Fixnum.
+ *         - An Integer.
  *
  * Returns the fd number >= 0 if one could be established, or -1 if the object
  * does not map to an fd.
@@ -56,7 +56,11 @@ posixspawn_obj_to_fd(VALUE obj)
 	int fd = -1;
 	switch (TYPE(obj)) {
 		case T_FIXNUM:
-			/* Fixnum fd number */
+		case T_BIGNUM:
+			/* Integer fd number
+			 * rb_fix2int takes care of raising if the provided object is a
+			 * Bignum and is out of range of an int
+			 */
 			fd = FIX2INT(obj);
 			break;
 
@@ -94,7 +98,7 @@ posixspawn_obj_to_fd(VALUE obj)
 /*
  * Hash iterator that sets up the posix_spawn_file_actions_t with addclose
  * operations. Only hash pairs whose value is :close are processed. Keys may
- * be the :in, :out, :err, an IO object, or a Fixnum fd number.
+ * be the :in, :out, :err, an IO object, or an Integer fd number.
  *
  * Returns ST_DELETE when an addclose operation was added; ST_CONTINUE when
  * no operation was performed.
@@ -279,6 +283,11 @@ each_env_i(VALUE key, VALUE val, VALUE arg)
 		const char *ev = envp[i];
 
 		if (strlen(ev) > name_len && !memcmp(ev, name, name_len) && ev[name_len] == '=') {
+			/* This operates on a duplicated environment -- release the
+			 * existing entry memory before shifting the subsequent entry
+			 * pointers down. */
+			free(envp[i]);
+
 			for (j = i; envp[j]; ++j)
 				envp[j] = envp[j + 1];
 			continue;

data/lib/posix/spawn/child.rb

@@ -1,5 +1,3 @@
-require 'posix/spawn'
-
 module POSIX
   module Spawn
     # POSIX::Spawn::Child includes logic for executing child processes and

data/lib/posix/spawn/version.rb

@@ -1,5 +1,5 @@
 module POSIX
   module Spawn
-    VERSION = '0.3.12'
+    VERSION = '0.3.15'
   end
 end

data/lib/posix/spawn.rb

@@ -85,7 +85,7 @@ module POSIX
   #
   #     spawn(command, :chdir => "/var/tmp")
   #
-  # The :in, :out, :err, a Fixnum, an IO object or an Array option specify
+  # The :in, :out, :err, an Integer, an IO object or an Array option specify
   # fd redirection. For example, stderr can be merged into stdout as follows:
   #
   #     spawn(command, :err => :out)
@@ -460,11 +460,11 @@ module POSIX
 
     # Determine whether object is fd-like.
     #
-    # Returns true if object is an instance of IO, Fixnum >= 0, or one of the
+    # Returns true if object is an instance of IO, Integer >= 0, or one of the
     # the symbolic names :in, :out, or :err.
     def fd?(object)
       case object
-      when Fixnum
+      when Integer
         object >= 0
       when :in, :out, :err, STDIN, STDOUT, STDERR, $stdin, $stdout, $stderr, IO
         true
@@ -486,7 +486,7 @@ module POSIX
         STDOUT
       when :err, 2
         STDERR
-      when Fixnum
+      when Integer
         object >= 0 ? IO.for_fd(object) : nil
       when IO
         object
@@ -529,7 +529,7 @@ module POSIX
     #
     # Returns a [[cmdname, argv0], argv1, ...] array.
     def adjust_process_spawn_argv(args)
-      if args.size == 1 && args[0] =~ /[ |>]/
+      if args.size == 1 && args[0].is_a?(String) && args[0] =~ /[ |>]/
         # single string with these characters means run it through the shell
         command_and_args = system_command_prefixes + [args[0]]
         [*command_and_args]

data/test/test_spawn.rb

@@ -322,7 +322,7 @@ module SpawnImplementationTests
       end
     end
 
-    assert_match /oops/, exception.message
+    assert_match(/oops/, exception.message)
   end
 
   ##

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: posix-spawn
 version: !ruby/object:Gem::Version
-  version: 0.3.12
+  version: 0.3.15
 platform: ruby
 authors:
 - Ryan Tomayko
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-03 00:00:00.000000000 Z
+date: 2020-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -52,6 +52,7 @@ extra_rdoc_files:
 - HACKING
 files:
 - ".gitignore"
+- ".travis.yml"
 - COPYING
 - Gemfile
 - HACKING
@@ -91,10 +92,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: posix_spawnp(2) for ruby
 test_files: []
-has_rdoc: