make-snapshot: deprecated -exported option [Bug #16167]

tool/make-snapshot

@@ -12,7 +12,6 @@ require File.expand_path("../lib/colorize", __FILE__)
 STDOUT.sync = true
 
 $srcdir ||= nil
-$exported = nil if ($exported ||= nil) == ""
 $archname = nil if ($archname ||= nil) == ""
 $keep_temp ||= nil
 $patch_file ||= nil
@@ -28,7 +27,6 @@ def usage
 usage: #{File.basename $0} [option...] new-directory-to-save [version ...]
 options:
   -srcdir=PATH          source directory path
-  -exported=PATH        make snapshot from already exported working directory
   -archname=NAME        make the basename of snapshots NAME
   -keep_temp            keep temporary working directory
   -patch_file=PATCH     apply PATCH file after export
@@ -143,16 +141,19 @@ unless destdir = ARGV.shift
   abort usage
 end
 revisions = ARGV.empty? ? [nil] : ARGV
-unless tmp = $exported
+
-  FileUtils.mkpath(destdir)
+if $exported
-  destdir = File.expand_path(destdir)
+  abort "#{File.basename $0}: -exported option is deprecated; use -srcdir instead"
-  tmp = Dir.mktmpdir("ruby-snapshot")
+end
-  FileUtils.mkpath(tmp)
+
-  at_exit {
+FileUtils.mkpath(destdir)
+destdir = File.expand_path(destdir)
+tmp = Dir.mktmpdir("ruby-snapshot")
+FileUtils.mkpath(tmp)
+at_exit {
   Dir.chdir "/"
   FileUtils.rm_rf(tmp)
-  } unless $keep_temp
+} unless $keep_temp
-end
 
 def tar_create(tarball, dir)
   require 'rubygems'
@@ -273,12 +274,7 @@ def package(vcs, rev, destdir, tmp = nil)
     end
     revision = vcs.get_revisions(url)[1]
   end
-  v = nil
+
-  if $exported
-    if String === $exported
-      v = $exported
-    end
-  else
   v = "ruby"
   puts "Exporting #{rev}@#{revision}"
   exported = tmp ? File.join(tmp, v) : v
@@ -303,7 +299,6 @@ def package(vcs, rev, destdir, tmp = nil)
       end
     end
   end
-  end
 
   status = IO.read(File.dirname(__FILE__) + "/prereq.status")
   Dir.chdir(tmp) if tmp
@@ -344,7 +339,7 @@ def package(vcs, rev, destdir, tmp = nil)
   else
     tag ||= vcs.revision_name(revision)
   end
-  unless v == $exported
+
   if $archname
     n = $archname
   elsif tag.empty?
@@ -354,7 +349,7 @@ def package(vcs, rev, destdir, tmp = nil)
   end
   File.directory?(n) or File.rename v, n
   vcs.chdir(File.expand_path(v = n))
-  end
+
   system(*%W"patch -d #{v} -p0 -i #{$patch_file}") if $patch_file
   def (clean = []).add(n) push(n); n end
   Dir.chdir(v) do
@@ -362,7 +357,7 @@ def package(vcs, rev, destdir, tmp = nil)
       vcs.export_changelog(url, nil, revision, "ChangeLog")
     end
 
-    if !$exported or $patch_file and !touch_all(modified, "**/*", File::FNM_DOTMATCH)
+    unless touch_all(modified, "**/*", File::FNM_DOTMATCH)
       modified = nil
       colors = %w[red yellow green cyan blue magenta]
       "take a breath, and go ahead".scan(/./) do |c|
@@ -553,7 +548,7 @@ touch-unicode-files:
     end
   end.compact
 ensure
-  FileUtils.rm_rf(tmp ? File.join(tmp, v) : v) if v and !$exported and !$keep_temp
+  FileUtils.rm_rf(tmp ? File.join(tmp, v) : v) if v and !$keep_temp
   Dir.chdir(pwd)
 end