Since now fetch_ref is reliable, we could just rely on it

This commit is contained in:
Lin Jen-Shin 2017-08-11 20:12:17 +08:00
parent 501be36c2e
commit ca685f80e0
1 changed files with 5 additions and 5 deletions

View File

@ -989,12 +989,10 @@ class Repository
if start_repository == self if start_repository == self
start_branch_name start_branch_name
else else
tmp_ref = "refs/tmp/#{SecureRandom.hex}/head" tmp_ref = fetch_ref(
fetch_ref(
start_repository.path_to_repo, start_repository.path_to_repo,
"#{Gitlab::Git::BRANCH_REF_PREFIX}#{start_branch_name}", "#{Gitlab::Git::BRANCH_REF_PREFIX}#{start_branch_name}",
tmp_ref "refs/tmp/#{SecureRandom.hex}/head"
) )
start_repository.commit(start_branch_name).sha start_repository.commit(start_branch_name).sha
@ -1003,7 +1001,7 @@ class Repository
yield(commit(branch_name_or_sha)) yield(commit(branch_name_or_sha))
ensure ensure
rugged.references.delete(tmp_ref) if tmp_ref && ref_exists?(tmp_ref) rugged.references.delete(tmp_ref) if tmp_ref
end end
def add_remote(name, url) def add_remote(name, url)
@ -1029,6 +1027,8 @@ class Repository
# Make sure ref was created, and raise Rugged::ReferenceError when not # Make sure ref was created, and raise Rugged::ReferenceError when not
raise Rugged::ReferenceError, message if status != 0 raise Rugged::ReferenceError, message if status != 0
target_ref
end end
def create_ref(ref, ref_path) def create_ref(ref, ref_path)