Commit 06154af0 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'satellite_compare_gitlab_git' into 'master'

Use Gitlab::Git::Compare in Satellite::CompareAction

See merge request !976
parents 0768100a 2ef45507
...@@ -17,11 +17,7 @@ module Gitlab ...@@ -17,11 +17,7 @@ module Gitlab
in_locked_and_timed_satellite do |target_repo| in_locked_and_timed_satellite do |target_repo|
prepare_satellite!(target_repo) prepare_satellite!(target_repo)
update_satellite_source_and_target!(target_repo) update_satellite_source_and_target!(target_repo)
common_commit = target_repo.git.native(:merge_base, default_options, ["origin/#{@target_branch}", "source/#{@source_branch}"]).strip compare(target_repo).diffs
#this method doesn't take default options
diffs = target_repo.diff(common_commit, "source/#{@source_branch}")
diffs = diffs.map { |diff| Gitlab::Git::Diff.new(diff) }
diffs
end end
rescue Grit::Git::CommandFailed => ex rescue Grit::Git::CommandFailed => ex
raise BranchesWithoutParent raise BranchesWithoutParent
...@@ -32,9 +28,7 @@ module Gitlab ...@@ -32,9 +28,7 @@ module Gitlab
in_locked_and_timed_satellite do |target_repo| in_locked_and_timed_satellite do |target_repo|
prepare_satellite!(target_repo) prepare_satellite!(target_repo)
update_satellite_source_and_target!(target_repo) update_satellite_source_and_target!(target_repo)
commits = target_repo.commits_between("origin/#{@target_branch}", "source/#{@source_branch}") compare(target_repo).commits
commits = commits.map { |commit| Gitlab::Git::Commit.new(commit, nil) }
commits
end end
rescue Grit::Git::CommandFailed => ex rescue Grit::Git::CommandFailed => ex
handle_exception(ex) handle_exception(ex)
...@@ -46,10 +40,13 @@ module Gitlab ...@@ -46,10 +40,13 @@ module Gitlab
def update_satellite_source_and_target!(target_repo) def update_satellite_source_and_target!(target_repo)
target_repo.remote_add('source', @source_project.repository.path_to_repo) target_repo.remote_add('source', @source_project.repository.path_to_repo)
target_repo.remote_fetch('source') target_repo.remote_fetch('source')
target_repo.git.checkout(default_options({b: true}), @target_branch, "origin/#{@target_branch}")
rescue Grit::Git::CommandFailed => ex rescue Grit::Git::CommandFailed => ex
handle_exception(ex) handle_exception(ex)
end end
def compare(repo)
@compare ||= Gitlab::Git::Compare.new(Gitlab::Git::Repository.new(repo.path), "origin/#{@target_branch}", "source/#{@source_branch}")
end
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment