Fix patch_for_go_gem

Author: sue445

_tools/patch_for_go_gem/patch_for_go_gem.rb

@@ -31,6 +31,7 @@ def perform
     create_go_mod
     update_gem_name_c
     update_extconf_rb
+    update_gemspec
   end
 
   private
@@ -125,65 +126,45 @@ def update_gem_name_c
     save_file(file_path: gem_name_c_path, content:)
   end
 
-  def update_extconf_rb # rubocop:disable Metrics/MethodLength
+  def update_extconf_rb
     extconf_rb_path = File.join(ext_dir, "extconf.rb")
 
     content = File.read(extconf_rb_path)
 
-    unless content.include?(<<~RUBY)
-      require "mkmf"
-
-      find_executable("go")
-    RUBY
-
+    unless content.include?(%(require "go_gem/mkmf"))
       content.gsub!(<<~RUBY, <<~RUBY)
         require "mkmf"
       RUBY
         require "mkmf"
-
-        find_executable("go")
-
-        # rubocop:disable Style/GlobalVars
-        $objs = []
-        def $objs.empty?; false; end
-        # rubocop:enable Style/GlobalVars
-
+        require "go_gem/mkmf"
       RUBY
     end
 
-    unless content.include?(<<~RUBY)
-      create_makefile("#{gem_name}/#{gem_name}")
-
-      case `\#{CONFIG["CC"]} --version` # rubocop:disable Lint/LiteralAsCondition
-    RUBY
-
+    unless content.include?(%(create_go_makefile("#{gem_name}/#{gem_name}")))
       content.gsub!(<<~RUBY, <<~RUBY)
         create_makefile("#{gem_name}/#{gem_name}")
       RUBY
-        create_makefile("#{gem_name}/#{gem_name}")
+        create_go_makefile("#{gem_name}/#{gem_name}")
+      RUBY
+    end
 
-        case `\#{CONFIG["CC"]} --version` # rubocop:disable Lint/LiteralAsCondition
-        when /Free Software Foundation/
-          ldflags = "-Wl,--unresolved-symbols=ignore-all"
-        when /clang/
-          ldflags = "-undefined dynamic_lookup"
-        end
+    save_file(file_path: extconf_rb_path, content:)
+  end
 
-        current_dir = File.expand_path(".")
+  def update_gemspec
+    content = File.read(gemspec_file)
 
-        File.open("Makefile", "a") do |f|
-          f.write <<~MAKEFILE.gsub(/^ {8}/, "\t")
-            $(DLLIB): Makefile $(srcdir)/*.go
-                    cd $(srcdir); \
-                    CGO_CFLAGS='$(INCFLAGS)' CGO_LDFLAGS='\#{ldflags}' \\
-                      go build -p 4 -buildmode=c-shared -o \#{current_dir}/$(DLLIB)
-          MAKEFILE
-        end
+    return if content.include?(%(.add_dependency "go_gem")) || content.include?(%(.add_runtime_dependency "go_gem"))
 
-      RUBY
-    end
+    content =~ /Gem::Specification\.new\s+do\s+\|(.+)\|/
+    spec_var_name = ::Regexp.last_match(1)
 
-    save_file(file_path: extconf_rb_path, content:)
+    content.gsub!(/^end\n/, <<~RUBY)
+        #{spec_var_name}.add_dependency "go_gem"
+      end
+    RUBY
+
+    save_file(file_path: gemspec_file, content:)
   end
 
   # @param file_path [String]

_tools/patch_for_go_gem/spec/patch_for_go_gem_spec.rb

@@ -88,47 +88,18 @@ module github.com/username/#{gem_name}
   end
 
   describe file(File.join(gem_name, "ext", gem_name, "extconf.rb")) do
-    let(:content1) do
-      <<~RUBY
-        require "mkmf"
-
-        find_executable("go")
-
-        # rubocop:disable Style/GlobalVars
-        $objs = []
-        def $objs.empty?; false; end
-        # rubocop:enable Style/GlobalVars
-      RUBY
-    end
-
-    let(:content2) do
-      <<~RUBY
-        create_makefile("#{gem_name}/#{gem_name}")
-
-        case `\#{CONFIG["CC"]} --version` # rubocop:disable Lint/LiteralAsCondition
-        when /Free Software Foundation/
-          ldflags = "-Wl,--unresolved-symbols=ignore-all"
-        when /clang/
-          ldflags = "-undefined dynamic_lookup"
-        end
-
-        current_dir = File.expand_path(".")
+    it { should be_file }
+    it { should exist }
 
-        File.open("Makefile", "a") do |f|
-          f.write <<~MAKEFILE.gsub(/^ {8}/, "\t")
-            $(DLLIB): Makefile $(srcdir)/*.go
-                    cd $(srcdir); \
-                    CGO_CFLAGS='$(INCFLAGS)' CGO_LDFLAGS='\#{ldflags}' \\
-                      go build -p 4 -buildmode=c-shared -o \#{current_dir}/$(DLLIB)
-          MAKEFILE
-        end
-      RUBY
-    end
+    its(:content) { should include %(require "go_gem/mkmf") }
+    its(:content) { should include %(create_go_makefile("#{gem_name}/#{gem_name}")) }
+    its(:content) { should_not include %(create_makefile("#{gem_name}/#{gem_name}")) }
+  end
 
+  describe file(File.join(gem_name, "#{gem_name}.gemspec")) do
     it { should be_file }
     it { should exist }
 
-    its(:content) { should include content1 }
-    its(:content) { should include content2 }
+    its(:content) { should include %(spec.add_dependency "go_gem") }
   end
 end