build: Print stdout only when --verbose is specified

Author: kateinoigakukun

.github/workflows/build.yml

@@ -121,7 +121,7 @@ jobs:
         run: rake ${{ matrix.entry.prerelease }}[${{ inputs.prerel_name }}]
         if: ${{ inputs.prerel_name != '' && matrix.entry.prerelease != '' }}
       - name: rake ${{ matrix.entry.task }}
-        run: docker run -v "$GITHUB_WORKSPACE:/home/me/build" -w /home/me/build -e "RUBYWASM_UID=$(id -u)" -e "RUBYWASM_GID=$(id -g)" -e "GNUMAKEFLAGS=-j$((1 + $(nproc --all)))" ${{ steps.builder-image.outputs.imageid }} rake ${{ matrix.entry.task }}
+        run: docker run -v "$GITHUB_WORKSPACE:/home/me/build" -w /home/me/build -e "RUBYWASM_UID=$(id -u)" -e "RUBYWASM_GID=$(id -g)" -e "GNUMAKEFLAGS=-j$((1 + $(nproc --all)))" ${{ steps.builder-image.outputs.imageid }} rake --verbose ${{ matrix.entry.task }}
       - name: rake ${{ matrix.entry.test }}
         run: rake ${{ matrix.entry.test }}
         if: ${{ matrix.entry.test != '' }}

lib/ruby_wasm/build.rb

@@ -5,8 +5,18 @@
 module RubyWasm
   # Build executor to run the actual build commands.
   class BuildExecutor
+    def initialize(verbose: false)
+      @verbose = verbose
+    end
+
     def system(*args, **kwargs)
-      Kernel.system(*args, **kwargs)
+      puts args.join(" ")
+      if @verbose
+        out = kwargs[:out] || $stdout
+      else
+        out = IO.pipe[1]
+      end
+      Kernel.system(*args, **kwargs, out: out)
     end
 
     def rm_rf(list)

lib/ruby_wasm/rake_task.rb

@@ -71,7 +71,8 @@ def initialize(
     @crossruby.with_wasi_vfs @wasi_vfs
     @crossruby.with_openssl @openssl
 
-    executor = RubyWasm::BuildExecutor.new
+    # Rake.verbose can be Object.new by default, so compare with true explicitly.
+    executor = RubyWasm::BuildExecutor.new(verbose: Rake.verbose == true)
 
     desc "Cross-build Ruby for #{@target}"
     task name do

sig/ruby_wasm/build.rbs

@@ -222,6 +222,9 @@ module RubyWasm
   end
 
   class BuildExecutor
+    @verbose: bool
+
+    def initialize: (?verbose: bool) -> void
     def system: (*untyped, **untyped) -> bool?
     def rm_rf: (FileUtils::pathlist list) -> void
     def rm_f: (FileUtils::pathlist list) -> void