Cache destination dir.

It is not necessary to strip the `destdir` prefix every iteration, when it can be done just once.
koic · Feb 26, 2020 · 9d6d531 · 9d6d531
1 parent b8a8fdb
commit 9d6d531
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/tool/rbinstall.rb b/tool/rbinstall.rb
@@ -722,9 +722,10 @@ def extract_files(destination_dir, pattern = "*")
       return if @src_dir == destination_dir
       File.chmod(0700, destination_dir)
       mode = pattern == File.join(spec.bindir, '*') ? prog_mode : data_mode
+      destdir = without_destdir(destination_dir)
       spec.files.each do |f|
         src = File.join(@src_dir, f)
-        dest = File.join(without_destdir(destination_dir), f)
+        dest = File.join(destdir, f)
         makedirs(dest[/.*(?=\/)/m])
         install src, dest, :mode => mode
       end