X-Git-Url: https://git.openstreetmap.org./chef.git/blobdiff_plain/48debd1ec47bbada97c4ef9ec7ae7e4daf3ded53..a0858dce1ce11c24de0103a341bc0ea8f75154ea:/cookbooks/chef/libraries/subversion.rb?ds=sidebyside diff --git a/cookbooks/chef/libraries/subversion.rb b/cookbooks/chef/libraries/subversion.rb index 25485f4bb..4f2b1cd00 100644 --- a/cookbooks/chef/libraries/subversion.rb +++ b/cookbooks/chef/libraries/subversion.rb @@ -13,23 +13,22 @@ class Chef end def current_repository - @current_repository ||= repo_attrs['URL'] + @current_repository ||= repo_attrs["URL"] end def current_repository_matches_target_repository? - (!current_repository.nil?) && (@new_resource.repository == current_repository) + !current_repository.nil? && (@new_resource.repository == current_repository) end def repo_attrs return {} unless ::File.exist?(::File.join(@new_resource.destination, ".svn")) - @repo_attrs ||= svn_info.lines.each_with_object({}) do |attrs, line| - if line =~ SVN_INFO_PATTERN - property, value = Regexp.last_match[1], Regexp.last_match[2] - attrs[property] = value - else - fail "Could not parse `svn info` data: #{line}" - end + @repo_attrs ||= svn_info.lines.each_with_object({}) do |line, attrs| + raise "Could not parse `svn info` data: #{line}" unless line =~ SVN_INFO_PATTERN + + property = Regexp.last_match[1] + value = Regexp.last_match[2] + attrs[property] = value end end