binaryen 1.1.6.6 → 1.1.6.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44bf5b373ce186bdee3006d16e8129bde2be3e952c1211d3ee550fb529cffbfe
-  data.tar.gz: c3ef7a60460cdc937e51b538ef4e6d94ebfba2167ff14821292881a9b91d6f1f
+  metadata.gz: e208c99114153cd5edae8ca1125d26eeecb8f2b5f91e218ccc63f807827ee4fe
+  data.tar.gz: 6839fedb4b0c710fc1aa852909f3a24daf1729a2f1531e6c3c47b7594f2819d3
 SHA512:
-  metadata.gz: 290897e4df9079e57df51b4b684fa5ae496ea414c44d801ed8d2be5c167d5347812533511ee26b8f59520d645df1072df5c6368d218ab824d89b38e07373e7b5
-  data.tar.gz: c30ed5adcb899a8d938d211b3aa3e98e875ec061f19054efd5d7a803b9be0392f9215cbac61283147841e23171ead48391f06695aa06841a64a6be05d5ba57aa
+  metadata.gz: 21ec4f01eacfa0d38b08acae723f5f6bf662f42047950354c3260b9d461b774ba91749b82b162a74b83ef0e70aa4b6541fcbbbb70111cd30b11b969955a2cfec
+  data.tar.gz: 5dbbf03fe6f8325cdaa1e78c81bfb7feef07d1887d53a647202f56dc58e5ac1b628a2057f0e5cce083bd2868894e622054f1a999d476a568ef04a5c1558a2d72

data/lib/binaryen/command.rb

@@ -15,29 +15,11 @@ module Binaryen
   #   optimized_wasm = command.run("-O4", stdin: "(module)")
   #   ```
   class Command
-    include POSIX::Spawn
-
+    MAX_OUTPUT_SIZE = 256 * 1024 * 1024
     DEFAULT_ARGS_FOR_COMMAND = {
       "wasm-opt" => ["--output=-"],
     }.freeze
 
-    class TimeoutChecker
-      def initialize(end_time:, pid:)
-        @end_time = end_time
-        @pid = pid
-      end
-
-      def check!
-        now = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-        if now >= @end_time
-          Process.kill("TERM", @pid)
-          raise Timeout::Error, "Command timed out"
-        end
-        remaining_time = @end_time - now
-        remaining_time
-      end
-    end
-
     def initialize(cmd, timeout: 10, ignore_missing: false)
       @cmd = command_path(cmd, ignore_missing) || raise(ArgumentError, "command not found: #{cmd}")
       @timeout = timeout
@@ -45,85 +27,22 @@ module Binaryen
     end
 
     def run(*arguments, stdin: nil, stderr: nil)
-      end_time = Process.clock_gettime(Process::CLOCK_MONOTONIC) + @timeout
-      command = build_command(*arguments)
-      pid, iwr, ord, erd = popen4(*command)
-      timeout_checker = TimeoutChecker.new(end_time: end_time, pid: pid)
-
-      write_to_pipe(iwr, stdin, timeout_checker, pid) if stdin
+      args = [@cmd] + arguments + @default_args
+      child = POSIX::Spawn::Child.new(*args, input: stdin, timeout: @timeout, max: MAX_OUTPUT_SIZE)
+      stderr&.write(child.err)
+      status = child.status
 
-      if stderr
-        err_output = read_from_pipe(erd, timeout_checker)
-        write_to_pipe(stderr, err_output, timeout_checker, pid, close_write: false)
-      end
-      output = read_from_pipe(ord, timeout_checker)
-      wait_or_kill(pid, timeout_checker)
+      raise Binaryen::NonZeroExitStatus, "command exited with status #{status.exitstatus}" unless status.success?
 
-      output
+      child.out
+    rescue POSIX::Spawn::MaximumOutputExceeded => e
+      raise Binaryen::MaximumOutputExceeded, e.message
+    rescue POSIX::Spawn::TimeoutExceeded => e
+      raise Timeout::Error, e.message
     end
 
     private
 
-    def write_to_pipe(pipe, stdin, timeout_checker, pid, close_write: true)
-      offset = 0
-      length = stdin.bytesize
-
-      while offset < length
-        remaining_time = timeout_checker.check!
-
-        next unless IO.select(nil, [pipe], nil, remaining_time)
-
-        begin
-          written = pipe.write_nonblock(stdin.byteslice(offset, length), exception: false)
-          offset += written if written.is_a?(Integer)
-        rescue Errno::EPIPE
-          wait_or_kill(pid, timeout_checker, pid)
-        end
-      end
-    ensure
-      pipe.close_write if close_write
-    end
-
-    def read_from_pipe(pipe, timeout_checker, close_read: true)
-      output = +""
-
-      while (result = pipe.read_nonblock(8192, exception: false))
-        remaining_time = timeout_checker.check!
-
-        case result
-        when :wait_readable
-          IO.select([pipe], nil, nil, remaining_time)
-        when nil
-          break
-        else
-          output << result
-        end
-      end
-
-      output
-    ensure
-      pipe.close_read if close_read
-    end
-
-    def wait_or_kill(pid, timeout_checker)
-      loop do
-        timeout_checker.check!
-
-        if (_, status = Process.wait2(pid, Process::WNOHANG))
-          raise Binaryen::NonZeroExitStatus,
-            "command exited with status #{status.exitstatus}" if status.exitstatus != 0
-
-          return true
-        else
-          sleep(0.1)
-        end
-      end
-    end
-
-    def build_command(*arguments)
-      [@cmd] + arguments + @default_args
-    end
-
     def command_path(cmd, ignore_missing)
       Dir[File.join(Binaryen.bindir, cmd)].first || (ignore_missing && cmd)
     end

data/lib/binaryen/error.rb

@@ -3,4 +3,5 @@
 module Binaryen
   class Error < StandardError; end
   class NonZeroExitStatus < Error; end
+  class MaximumOutputExceeded < Error; end
 end

data/lib/binaryen/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: binaryen
 version: !ruby/object:Gem::Version
-  version: 1.1.6.6
+  version: 1.1.6.7
 platform: ruby
 authors:
 - Shopify Inc.