diff options
author | Benoit Daloze <eregontp@gmail.com> | 2021-10-20 21:41:45 +0200 |
---|---|---|
committer | Benoit Daloze <eregontp@gmail.com> | 2021-10-20 21:41:45 +0200 |
commit | 207a5a5bc13018344dc2ab7913fdcaeaeca01292 (patch) | |
tree | a4c83bb4affa1dad3a4c004b4ecb88c0183aaf9b | |
parent | 5322745b29b23d4776345ee5bfa3a976497b49ee (diff) | |
download | ruby-207a5a5bc13018344dc2ab7913fdcaeaeca01292.tar.gz |
Update to ruby/mspec@08e1275
-rw-r--r-- | spec/mspec/tool/sync/sync-rubyspec.rb | 9 | ||||
-rwxr-xr-x | spec/mspec/tool/tag_from_output.rb | 7 |
2 files changed, 13 insertions, 3 deletions
diff --git a/spec/mspec/tool/sync/sync-rubyspec.rb b/spec/mspec/tool/sync/sync-rubyspec.rb index 7f3dc0e611..b4c79d2afc 100644 --- a/spec/mspec/tool/sync/sync-rubyspec.rb +++ b/spec/mspec/tool/sync/sync-rubyspec.rb @@ -57,6 +57,10 @@ class RubyImplementation File.basename(git_url, ".git") end + def repo_path + "#{__dir__}/#{repo_name}" + end + def repo_org File.basename(File.dirname(git_url)) end @@ -152,6 +156,11 @@ def rebase_commits(impl) raise "#{days_since_last_merge.floor} days since last merge, probably wrong commit" end + puts "Checking if the last merge is consistent with upstream files" + rubyspec_commit = `git log -n 1 --format='%s' #{last_merge}`.chomp.split('@', 2)[-1] + sh "git", "checkout", last_merge + sh "git", "diff", "--exit-code", rubyspec_commit, "--", ":!.github" + puts "Rebasing..." sh "git", "branch", "-D", rebased if branch?(rebased) sh "git", "checkout", "-b", rebased, impl.name diff --git a/spec/mspec/tool/tag_from_output.rb b/spec/mspec/tool/tag_from_output.rb index fba52ec26c..ebe13434c2 100755 --- a/spec/mspec/tool/tag_from_output.rb +++ b/spec/mspec/tool/tag_from_output.rb @@ -15,9 +15,10 @@ NUMBER = /^\d+\)$/ ERROR_OR_FAILED = / (ERROR|FAILED)$/ SPEC_FILE = /^(\/.+_spec\.rb)\:\d+/ -output.slice_before(NUMBER).select { |number, error_line, *rest| - number =~ NUMBER and error_line =~ ERROR_OR_FAILED -}.each { |number, error_line, *rest| +output.slice_before(NUMBER).select { |number, *rest| + number =~ NUMBER and rest.any? { |line| line =~ ERROR_OR_FAILED } +}.each { |number, *rest| + error_line = rest.find { |line| line =~ ERROR_OR_FAILED } description = error_line.match(ERROR_OR_FAILED).pre_match spec_file = rest.find { |line| line =~ SPEC_FILE } |