Bypass signal handling in interpreted code

.github/workflows/interpreter.yml

@@ -42,6 +42,7 @@ jobs:
   test-interpreter-std_spec:
     needs: build-interpreter
     runs-on: ubuntu-22.04
+    timeout-minutes: 15
     container:
       image: crystallang/crystal:1.11.2-build
     strategy:

spec/std/spec_helper.cr

@@ -108,12 +108,17 @@ def compile_source(source, flags = %w(), file = __FILE__, &)
 end
 
 def compile_and_run_file(source_file, flags = %w(), runtime_args = %w(), file = __FILE__)
-  compile_file(source_file, flags: flags, file: file) do |executable_file|
-    output, error = IO::Memory.new, IO::Memory.new
-    status = Process.run executable_file, args: runtime_args, output: output, error: error
-
-    {status, output.to_s, error.to_s}
-  end
+  status, output, error = Process::Status.new(0), IO::Memory.new, IO::Memory.new
+  {% if flag?(:interpreted) %}
+    compiler = ENV["CRYSTAL_SPEC_COMPILER_BIN"]? || "bin/crystal"
+    args = ["i", source_file, "--", *runtime_args]
+    status = Process.run compiler, args: args, output: output, error: error
+  {% else %}
+    compile_file(source_file, flags: flags, file: file) do |executable_file|
+      status = Process.run executable_file, args: runtime_args, output: output, error: error
+    end
+  {% end %}
+  {status, output.to_s, error.to_s}
 end
 
 def compile_and_run_source(source, flags = %w(), runtime_args = %w(), file = __FILE__)

spec/std/time/format_spec.cr

@@ -1,4 +1,5 @@
 require "../spec_helper"
+require "../../support/time"
 require "spec/helpers/string"
 
 def parse_time(format, string)

spec/std/time/time_spec.cr

@@ -1,4 +1,5 @@
 require "../spec_helper"
+require "../../support/time"
 require "spec/helpers/iterate"
 
 CALENDAR_WEEK_TEST_DATA = [

src/crystal/system/unix/process.cr

@@ -15,6 +15,24 @@ struct Crystal::System::Process
   end
 
   def wait
+    {% if flag?(:interpreted) %}
+      exit_code = uninitialized Int32
+      loop do
+        r_pid = LibC.waitpid(pid, pointerof(exit_code), LibC::WNOHANG)
+        case r_pid
+        when 0
+          sleep 1
+        when -1
+          raise RuntimeError.from_errno("waitpid") unless Errno.value == Errno::ECHILD
+          sleep 1
+        else
+          break
+        end
+      end
+
+      @channel.send(exit_code)
+      @channel.close
+    {% end %}
     @channel.receive
   end
 

src/crystal/system/unix/signal.cr

@@ -228,6 +228,13 @@ module Crystal::System::SignalChildHandler
 
   def self.wait(pid : LibC::PidT) : Channel(Int32)
     channel = Channel(Int32).new(1)
+    {% if flag?(:interpreted) %}
+      # Returns channel immediately when interpreted
+      # Waiting for child process in interpreted code
+      # does not rely on signal SIGCHLD and it's handled
+      # in ::Crystal::System::Process.wait
+      return channel
+    {% end %}
 
     @@mutex.lock
     if exit_code = @@pending.delete(pid)