binaryen 1.1.6.8 → 1.1.6.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4217d4168d453c8e5316e9c66929d5e09057ec356dc6f80802c41521de6fd612
-  data.tar.gz: 2cb4b068c0456d72dcc55cf0d945a5357df0d3ad8c82166fca8a57dcd2908ec4
+  metadata.gz: 7a03d6e05a577d6579506e64e603e11f9430bf4d4f19c4d3f5cf0e984ad78387
+  data.tar.gz: c2995a3d92c07731d5b54b832c0d838159aebd2770dc1b4e0155bcc09ef2e7a8
 SHA512:
-  metadata.gz: 9e3fd947316b466a8d592014bc57b1c34148da5b3042e0630b2bc5f56e54c287ab4c6416e78c1ef392e88429cbbcba0c0c3549bdd8b99d7068972c1adeb0c698
-  data.tar.gz: 2cdfe8e57ab692f16e911e7a819cb6cfecbf9b31027a04c8863eb919fd952cb6d92832d3884768cf05704507c7c408bcfb19eed3c47b90dfcbb6f1e7eaefc679
+  metadata.gz: 01cdacd58083894fd4356f23032b00c00a4f586ac4d141742a50471228d772623027178e2b93879d9d224b3dc98fcdeb5c90374700914a7021752acbdaa78720
+  data.tar.gz: cfddf72d19ad3922e7152ec150d1ce31a936b54f25e1638674be47d453dd808353110ec481749977bd54a78369b2db3c7331342af7ae918a9cbef447261ec3f9

data/lib/binaryen/command.rb

@@ -7,11 +7,12 @@ require "tempfile"
 module Binaryen
   class Command
     include POSIX::Spawn
-    DEFAULT_MAX_OUTPUT_SIZE = 256 * 1024 * 1024
+    DEFAULT_MAX_OUTPUT_SIZE = 256 * 1024 * 1024 * 1024 # 256 MiB
     DEFAULT_TIMEOUT = 10
     DEFAULT_ARGS_FOR_COMMAND = {
       "wasm-opt" => ["--output=-"],
     }.freeze
+    BUFSIZE = 32 * 1024
 
     def initialize(cmd, timeout: DEFAULT_TIMEOUT, max_output_size: DEFAULT_MAX_OUTPUT_SIZE, ignore_missing: false)
       @cmd = command_path(cmd, ignore_missing) || raise(ArgumentError, "command not found: #{cmd}")
@@ -47,11 +48,9 @@ module Binaryen
 
     def spawn_command(*args, stderr: nil)
       out = "".b
-      data_buffer = "".b
       start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       pid, stdin, stdout, stderr_stream = popen4(*args)
       stdin.close
-      @pid = pid
       readers = [stdout, stderr_stream]
 
       while readers.any?
@@ -61,33 +60,31 @@ module Binaryen
         raise Timeout::Error, "command timed out after #{@timeout} seconds" if ready.nil?
 
         ready[0].each do |io|
-          max_amount_to_read = @max_output_size - out.bytesize + 1
-          data = io.read_nonblock(max_amount_to_read, data_buffer, exception: false)
+          data = io.read_nonblock(BUFSIZE, exception: false)
 
           if data == :wait_readable
             # If the IO object is not ready for reading, read_nonblock returns :wait_readable.
             # This isn't an error, but a notification.
-            next
           elsif data.nil?
             # At EOF, read_nonblock returns nil instead of raising EOFError.
             readers.delete(io)
+            io.close
           elsif io == stdout
-            out << data_buffer
+            out << data
           elsif io == stderr_stream && stderr
-            stderr << data_buffer
+            stderr << data
           end
-        rescue Errno::EPIPE, Errno::EINTR
-          # Handle EPIPE and EINTR errors
-          readers.delete(io)
+        rescue Errno::EINTR
+          # This means that the read was interrupted by a signal, which is not an error. So we just retry.
         end
 
         if out.bytesize > @max_output_size
-          Process.kill("TERM", @pid)
+          Process.kill("TERM", pid)
           raise Binaryen::MaximumOutputExceeded, "maximum output size exceeded (#{@max_output_size} bytes)"
         end
 
         if remaining_time < 0
-          Process.kill("TERM", @pid)
+          Process.kill("TERM", pid)
           raise Timeout::Error, "command timed out after #{@timeout} seconds"
         end
       end

data/lib/binaryen/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Binaryen
-  VERSION = "1.1.6.8"
+  VERSION = "1.1.6.10"
   BINARYEN_VERSION = "version_116"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: binaryen
 version: !ruby/object:Gem::Version
-  version: 1.1.6.8
+  version: 1.1.6.10
 platform: ruby
 authors:
 - Shopify Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-12-19 00:00:00.000000000 Z
+date: 2024-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: posix-spawn
@@ -106,7 +106,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.22
+rubygems_version: 3.5.4
 signing_key: 
 specification_version: 4
 summary: Vendors binaryen libraries, headers, and executables for use in Ruby