Refactor logging to use BuilderOpsLogger module

- Moved the logging implementation from the Builder class to the BuilderOpsLogger module.
- Modified the Builder class to prepend the BuilderOpsLogger module when build logging is needed, enhancing modularity and separation of concerns.

bin/ocran

@@ -766,8 +766,10 @@ EOF
       builder.exec(installed_ruby_exe, target_script, *Ocran.arg)
     end
 
+    require_relative "../lib/ocran/builder_ops_logger"
     unless Ocran.inno_script
       require_relative "../lib/ocran/stub_builder"
+      StubBuilder.prepend(BuilderOpsLogger) unless Ocran.quiet
       StubBuilder.new(Ocran.output_executable,
                       chdir_before: Ocran.chdir_first,
                       debug_extract: Ocran.debug_extract,
@@ -779,6 +781,7 @@ EOF
       Ocran.msg "Finished building #{Ocran.output_executable} (#{Ocran.output_executable.size} bytes)"
     else
       require_relative "../lib/ocran/inno_setup_builder"
+      InnoSetupBuilder.prepend(BuilderOpsLogger) unless Ocran.quiet
       InnoSetupBuilder.new(Ocran.output_executable,
                            Ocran.inno_script,
                            chdir_before: Ocran.chdir_first,

lib/ocran/builder_ops_logger.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Ocran
+  module BuilderOpsLogger
+    def mkdir(target)
+      puts "mkdir #{target}"
+      super
+    end
+
+    def cp(source, target)
+      puts "cp #{source} #{target}"
+      super
+    end
+
+    def touch(target)
+      puts "touch #{target}"
+      super
+    end
+
+    def exec(image, script, *argv)
+      args = argv.map { |s| replace_placeholder(s) }.join(" ")
+      puts "exec #{image} #{script} #{args}"
+      super
+    end
+
+    def export(name, value)
+      puts "export #{name}=#{replace_placeholder(value)}"
+      super
+    end
+
+    def replace_placeholder(s)
+      s.to_s.gsub(TEMPDIR_ROOT.to_s, "<tempdir>")
+    end
+    private :replace_placeholder
+  end
+end

lib/ocran/inno_setup_builder.rb

@@ -48,12 +48,10 @@ def compile
 
     def mkdir(target)
       @iss.mkdir(target)
-      Ocran.verbose_msg "m #{target}"
     end
 
     def cp(source, target)
       @iss.cp(source, target)
-      Ocran.verbose_msg "a #{target}"
     end
 
     def touch(target)
@@ -69,18 +67,10 @@ def exec(image, script, *argv)
       @script_info = true
 
       @launcher.exec(image, script, *argv)
-      extra_argc = argv.map { |arg| quote_and_escape(arg) }.join(" ")
-      Ocran.verbose_msg "p #{image} #{script} #{show_path extra_argc}"
     end
 
     def export(name, value)
       @launcher.export(name, value)
-      Ocran.verbose_msg "e #{name} #{show_path value}"
     end
-
-    def show_path(x)
-      x.to_s.gsub(TEMPDIR_ROOT.to_s, "{app}")
-    end
-    private :show_path
   end
 end

lib/ocran/stub_builder.rb

@@ -102,7 +102,6 @@ def mkdir(path)
       return if @paths[key]
 
       @paths[key] = path
-      Ocran.verbose_msg "m #{path}"
 
       write_opcode(OP_CREATE_DIRECTORY)
       write_path(path)
@@ -118,7 +117,6 @@ def cp(src, tgt)
 
       @files[key] = [tgt, src]
 
-      Ocran.verbose_msg "a #{tgt}"
       write_opcode(OP_CREATE_FILE)
       write_path(tgt)
       write_file(src)
@@ -141,15 +139,11 @@ def exec(image, script, *argv)
       end
       @script_set = true
 
-      extra_argc = argv.map { |arg| "\"#{arg.gsub("\"", "\\\"")}\"" }.join(" ")
-
-      Ocran.verbose_msg "p #{image} #{script} #{show_path extra_argc}"
       write_opcode(OP_SET_SCRIPT)
       write_string_array(convert_to_native(image), convert_to_native(script), *argv)
     end
 
     def export(name, value)
-      Ocran.verbose_msg "e #{name} #{show_path value}"
       write_opcode(OP_SETENV)
       write_string(name.to_s)
       write_string(value.to_s)
@@ -160,11 +154,6 @@ def close
       @of.close
     end
 
-    def show_path(x)
-      x.to_s.gsub(TEMPDIR_ROOT.to_s, "<tempdir>")
-    end
-    private :show_path
-
     def compress
       IO.popen([LZMA_PATH, "e", "-si", "-so"], "r+b") do |lzma|
         _of, @of = @of, lzma