X-Git-Url: https://git.openstreetmap.org./chef.git/blobdiff_plain/ab47927c016597f8602b81dcdc168ad5c335389d..5a207c66f87d1d119e0b3824345e1371e686eb7c:/cookbooks/chef/libraries/subversion.rb?ds=inline diff --git a/cookbooks/chef/libraries/subversion.rb b/cookbooks/chef/libraries/subversion.rb index 8e9f1e3d2..a2a6e2ba9 100644 --- a/cookbooks/chef/libraries/subversion.rb +++ b/cookbooks/chef/libraries/subversion.rb @@ -5,6 +5,12 @@ class Chef class Subversion extend Chef::Mixin::ShellOut + def shell_out!(*args, **options) + options = args.pop if options.empty? && args.last.is_a?(Hash) + + super(*args, **options) + end + def sync_command if current_repository_matches_target_repository? c = scm :update, new_resource.svn_arguments, verbose, authentication, proxy, "-r#{revision_int}", new_resource.destination @@ -42,20 +48,18 @@ class Chef def svn_info command = scm(:info) - shell_out!(command, run_options(:cwd => cwd, :returns => [0, 1])).stdout + shell_out!(command, **run_options(:cwd => cwd, :returns => [0, 1])).stdout end def revision_int - @revision_int ||= begin - if new_resource.revision =~ /^\d+$/ - new_resource.revision - else - command = scm(:info, new_resource.repository, new_resource.svn_info_args, authentication, "-r#{new_resource.revision}") - svn_info = shell_out!(command, run_options(:returns => [0, 1])).stdout - - extract_revision_info(svn_info) - end - end + @revision_int ||= if new_resource.revision =~ /^\d+$/ + new_resource.revision + else + command = scm(:info, new_resource.repository, new_resource.svn_info_args, authentication, "-r#{new_resource.revision}") + svn_info = shell_out!(command, **run_options(:returns => [0, 1])).stdout + + extract_revision_info(svn_info) + end end end end