Remove the chdir feature of #command

smortex · smortex · commit 22f93c85ed81 · 2018-06-04T10:48:46.000+02:00
This argument was not used, so no functional change.
diff --git a/lib/git/lib.rb b/lib/git/lib.rb
@@ -37,7 +37,7 @@ def init(opts={})
       arr_opts = []
       arr_opts << '--bare' if opts[:bare]
 
-      command('init', arr_opts, chdir: false)
+      command('init', arr_opts)
     end
 
     # tries to clone the given repo
@@ -132,7 +132,7 @@ def log_commits(opts={})
 
       arr_opts += log_path_options(opts)
 
-      command_lines('log', arr_opts, chdir: true).map { |l| l.split.first }
+      command_lines('log', arr_opts).map { |l| l.split.first }
     end
 
     def full_log_commits(opts={})
@@ -143,7 +143,7 @@ def full_log_commits(opts={})
 
       arr_opts += log_path_options(opts)
 
-      full_log = command_lines('log', arr_opts, chdir: true)
+      full_log = command_lines('log', arr_opts)
 
       process_commit_log_data(full_log)
     end
@@ -411,7 +411,7 @@ def ls_files(location=nil)
     def ls_remote(location=nil)
       location ||= '.'
       Hash.new{ |h,k| h[k] = {} }.tap do |hsh|
-        command_lines('ls-remote', [location], chdir: false).each do |line|
+        command_lines('ls-remote', [location]).each do |line|
           (sha, info) = line.split("\t")
           (ref, type, name) = info.split('/', 3)
           type ||= 'head'
@@ -450,7 +450,7 @@ def config_get(name)
     end
 
     def global_config_get(name)
-      command('config', ['--global', '--get', name], chdir: false)
+      command('config', ['--global', '--get', name])
     end
 
     def config_list
@@ -466,7 +466,7 @@ def config_list
     end
 
     def global_config_list
-      parse_config_list command_lines('config', ['--global', '--list'], chdir: false)
+      parse_config_list command_lines('config', ['--global', '--list'])
     end
 
     def parse_config_list(lines)
@@ -479,7 +479,7 @@ def parse_config_list(lines)
     end
 
     def parse_config(file)
-      parse_config_list command_lines('config', ['--list', '--file', file], chdir: false)
+      parse_config_list command_lines('config', ['--list', '--file', file])
     end
 
     # Shows objects
@@ -502,7 +502,7 @@ def config_set(name, value)
     end
 
     def global_config_set(name, value)
-      command('config', ['--global', name, value], chdir: false)
+      command('config', ['--global', name, value])
     end
 
     # updates the repository index using the working directory content
@@ -671,10 +671,10 @@ def unmerged
     def conflicts # :yields: file, your, their
       self.unmerged.each do |f|
         your = Tempfile.new("YOUR-#{File.basename(f)}").path
-        command('show', ":2:#{f}", chdir: true, redirect: "> #{escape your}")
+        command('show', ":2:#{f}", redirect: "> #{escape your}")
 
         their = Tempfile.new("THEIR-#{File.basename(f)}").path
-        command('show', ":3:#{f}", chdir: true, redirect: "> #{escape their}")
+        command('show', ":3:#{f}", redirect: "> #{escape their}")
         yield(f, your, their)
       end
     end
@@ -799,7 +799,7 @@ def commit_tree(tree, opts = {})
       arr_opts << tree
       arr_opts << '-p' << opts[:parent] if opts[:parent]
       arr_opts += [opts[:parents]].map { |p| ['-p', p] }.flatten if opts[:parents]
-      command('commit-tree', arr_opts, chdir: true, redirect: "< #{escape t.path}")
+      command('commit-tree', arr_opts, redirect: "< #{escape t.path}")
     end
 
     def update_ref(branch, commit)
@@ -845,13 +845,13 @@ def archive(sha, file = nil, opts = {})
       arr_opts << "--remote=#{opts[:remote]}" if opts[:remote]
       arr_opts << sha
       arr_opts << '--' << opts[:path] if opts[:path]
-      command('archive', arr_opts, chdir: true, redirect: (opts[:add_gzip] ? '| gzip' : '') + " > #{escape file}")
+      command('archive', arr_opts, redirect: (opts[:add_gzip] ? '| gzip' : '') + " > #{escape file}")
       return file
     end
 
     # returns the current version of git, as an Array of Fixnums.
     def current_command_version
-      output = command('version', [], chdir: false)
+      output = command('version', [])
       version = output[/\d+\.\d+(\.\d+)+/]
       version.split('.').collect {|i| i.to_i}
     end
@@ -923,15 +923,15 @@ def with_custom_env_variables(&block)
     end
 
     def command(cmd, *opts, &block)
-      command_opts = { chdir: true, redirect: '' }
+      command_opts = { redirect: '' }
       if opts.last.is_a?(Hash)
         command_opts.merge!(opts.pop)
       end
       command_opts.keys.each do |k|
-        raise ArgumentError.new("Unsupported option: #{k}") unless %i[chdir redirect].include?(k)
+        raise ArgumentError.new("Unsupported option: #{k}") unless %i[redirect].include?(k)
       end
 
-      default_command_opts = { chdir: true, redirect: '' }
+      default_command_opts = { redirect: '' }
       command_opts = default_command_opts.merge(command_opts)
 
       global_opts = []
