commit: 0b92ce7546729284ffdca21d5e6b1658b187b870 Author: Michał Górny <mgorny <AT> gentoo <DOT> org> AuthorDate: Fri Oct 3 21:19:39 2014 +0000 Commit: Michał Górny <mgorny <AT> gentoo <DOT> org> CommitDate: Thu May 26 08:58:08 2016 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0b92ce75
git-r3.eclass: Remove unnecessary HEAD->branch mapping code, #524146 Remove the code responsible for recognizing which branch HEAD pointed out to since it was unsafe and unnecessarily complex. A proper match is not really necessary since all operations can be safely performed on an opaque 'HEAD' (or rather refs/git-r3/HEAD since fetching to HEAD directly breaks git a bit). eclass/git-r3.eclass | 62 ++++------------------------------------------------ 1 file changed, 4 insertions(+), 58 deletions(-) diff --git a/eclass/git-r3.eclass b/eclass/git-r3.eclass index 957ff08..48249d2 100644 --- a/eclass/git-r3.eclass +++ b/eclass/git-r3.eclass @@ -434,42 +434,6 @@ _git-r3_is_local_repo() { [[ ${uri} == file://* || ${uri} == /* ]] } -# @FUNCTION: _git-r3_find_head -# @USAGE: <head-ref> -# @INTERNAL -# @DESCRIPTION: -# Given a ref to which remote HEAD was fetched, try to find -# a branch matching the commit. Expects 'git show-ref' -# or 'git ls-remote' output on stdin. -_git-r3_find_head() { - debug-print-function ${FUNCNAME} "$@" - - local head_ref=${1} - local head_hash=$(git rev-parse --verify "${1}" || die) - local matching_ref - - # TODO: some transports support peeking at symbolic remote refs - # find a way to use that rather than guessing - - # (based on guess_remote_head() in git-1.9.0/remote.c) - local h ref - while read h ref; do - # look for matching head - if [[ ${h} == ${head_hash} ]]; then - # either take the first matching ref, or master if it is there - if [[ ! ${matching_ref} || ${ref} == refs/heads/master ]]; then - matching_ref=${ref} - fi - fi - done - - if [[ ! ${matching_ref} ]]; then - die "Unable to find a matching branch for remote HEAD (${head_hash})" - fi - - echo "${matching_ref}" -} - # @FUNCTION: git-r3_fetch # @USAGE: [<repo-uri> [<remote-ref> [<local-id>]]] # @DESCRIPTION: @@ -651,26 +615,10 @@ git-r3_fetch() { set -- "${fetch_command[@]}" echo "${@}" >&2 if "${@}"; then - if [[ ${clone_type} == mirror ]]; then - # find remote HEAD and update our HEAD properly - git symbolic-ref HEAD \ - "$(_git-r3_find_head refs/git-r3/HEAD \ - < <(git show-ref --heads || die))" \ - || die "Unable to update HEAD" - else # single or shallow - if [[ ${fetch_l} == HEAD ]]; then - # find out what branch we fetched as HEAD - local head_branch=$(_git-r3_find_head \ - refs/git-r3/HEAD \ - < <(git ls-remote --heads "${r}" || die)) - - # and move it to its regular place - git update-ref --no-deref "${head_branch}" \ - refs/git-r3/HEAD \ - || die "Unable to sync HEAD branch ${head_branch}" - git symbolic-ref HEAD "${head_branch}" \ + if [[ ${clone_type} == mirror || ${fetch_l} == HEAD ]]; then + # update our HEAD to match our remote HEAD ref + git symbolic-ref HEAD refs/git-r3/HEAD \ || die "Unable to update HEAD" - fi fi # now let's see what the user wants from us @@ -806,9 +754,7 @@ git-r3_checkout() { # setup 'alternates' to avoid copying objects echo "${orig_repo}/objects" > "${GIT_DIR}"/objects/info/alternates || die # now copy the refs - # [htn]* safely catches heads, tags, notes without complaining - # on non-existing ones, and omits internal 'git-r3' ref - cp -R "${orig_repo}"/refs/[htn]* "${GIT_DIR}"/refs/ || die + cp -R "${orig_repo}"/refs/* "${GIT_DIR}"/refs/ || die # (no need to copy HEAD, we will set it via checkout)