Update default gemspecs at preparing sources if possible

Author: nobu

common.mk

@@ -1522,6 +1522,7 @@ refresh-gems: update-bundled_gems prepare-gems
 prepare-gems: $(HAVE_BASERUBY:yes=update-gems) $(HAVE_BASERUBY:yes=extract-gems)
 extract-gems: $(HAVE_BASERUBY:yes=update-gems) $(HAVE_BASERUBY:yes=outdate-bundled-gems)
 update-gems: $(HAVE_BASERUBY:yes=outdate-bundled-gems)
+update-gems: $(HAVE_BASERUBY:yes=update-default-gemspecs)
 
 split_option = -F"\s+|$(HASH_SIGN).*"
 
@@ -1579,27 +1580,10 @@ no-test-bundled-gems-precheck:
 yes-update-default-gemspecs no-update-default-gemspecs: update-default-gemspecs
 update-default-gemspecs: $(PREP)
 	@$(MAKEDIRS) $(srcdir)/.bundle/specifications
-	$(Q)$(MINIRUBY) -W0 -C "$(srcdir)" -rrubygems \
-	    -e "destdir = ARGV.shift" \
-	    -e "ARGV.each do |basedir|" \
-	    -e   "Dir.glob(basedir+'/**/*.gemspec') do |g|" \
-	    -e     "dir, base = File.split(g)" \
-	    -e     "spec = Dir.chdir(dir) {Gem::Specification.load(base)} ||" \
-	    -e         "Gem::Specification.load(g)" \
-	    -e     "unless spec" \
-	    -e       "puts %[Ignoring #{g}]" \
-	    -e       "next" \
-	    -e     "end" \
-	    -e     "spec.files.clear" \
-	    -e     "spec.extensions.clear" \
-	    -e     "src = spec.to_ruby" \
-	    -e     "src.sub!(/^$$/) {" \
-	    -e       "%[# default: #{g} #{File.mtime(g).strftime(%[%s.%N])}\n]" \
-	    -e     "}" \
-	    -e     "File.binwrite(File.join(destdir, spec.full_name+'.gemspec'), src)" \
-	    -e   "end" \
-	    -e "end" \
-	    -- .bundle/specifications lib ext
+	$(Q)$(MINIRUBY) -W0 -C "$(srcdir)" -I tool/lib -roptparse -routput -rbundled_gem \
+	    -e "(out = Output.new).def_options(ARGV.options)" \
+	    -e "BundledGem.update_default_gemspecs(ARGV.parse!, out, quiet: $(V).zero?)" \
+	    -- -c -o .bundle/specifications lib ext
 
 install-for-test-bundled-gems: $(TEST_RUNNABLE)-install-for-test-bundled-gems
 no-install-for-test-bundled-gems: no-update-default-gemspecs

tool/lib/bundled_gem.rb

@@ -130,4 +130,30 @@ def checkout(gemdir, repo, rev, git: $git)
     command = "#{git} checkout --detach #{rev}"
     system(command, chdir: gemdir) or raise "failed: #{command}"
   end
+
+  def load_gemspec(g)
+    dir, base = File.split(g)
+    spec = Dir.chdir(dir) {Gem::Specification.load(base)} || Gem::Specification.load(g) or
+      return false
+    spec.files.clear
+    spec.extensions.clear
+    src = spec.to_ruby
+    src.sub!(/^$$/) {
+      %[# default: #{g} #{File.mtime(g).strftime(%[%s.%N])}\n]
+    }
+    return spec.full_name+'.gemspec', src
+  end
+
+  def update_default_gemspecs(basedirs, out, quiet: true)
+    basedirs.each do |basedir|
+      Dir.glob(basedir+'/**/*.gemspec') do |g|
+        name, src = BundledGem.load_gemspec(g)
+        unless src
+          puts "Ignoring #{g}" unless quiet
+          next
+        end
+        out.write(src, name: name, quiet: quiet)
+      end
+    end
+  end
 end

tool/lib/output.rb

@@ -31,30 +31,30 @@ def def_options(opt)
     @vpath.def_options(opt)
   end
 
-  def write(data, overwrite: @overwrite, create_only: @create_only)
-    unless @path
+  def write(data, overwrite: @overwrite, create_only: @create_only, name: nil, quiet: false)
+    outpath = name ? @path ? File.join(@path, name) : name : @path
+    unless outpath
       $stdout.print data
       return true
     end
     color = Colorize.new(@color)
     unchanged = color.pass("unchanged")
     updated = color.fail("updated")
-    outpath = nil
 
-    if (@ifchange or overwrite or create_only) and (@vpath.open(@path, "rb") {|f|
+    if (@ifchange or overwrite or create_only) and (@vpath.open(outpath, "rb") {|f|
       outpath = f.path
       if @ifchange or create_only
         original = f.read
         (@ifchange and original == data) or (create_only and !original.empty?)
       end
     } rescue false)
-      puts "#{outpath} #{unchanged}"
+      puts "#{outpath} #{unchanged}" unless quiet
       written = false
     else
       unless overwrite and outpath and (File.binwrite(outpath, data) rescue nil)
-        File.binwrite(outpath = @path, data)
+        File.binwrite(outpath, data)
       end
-      puts "#{outpath} #{updated}"
+      puts "#{outpath} #{updated}" unless quiet
       written = true
     end
     if timestamp = @timestamp