Re: [PATCH] t3400 (rebase): add failing test for a peculiar rev spec

Ramkumar Ramachandra [off-list ref] writes:

Junio C Hamano wrote:

quoted

        peel_committish () {
                case "$1" in
                :/*)
                        peeltmp=$(git rev-parse --verify "$1") &&
                        git rev-parse --verify "$peeltmp^0"
                        ;;
                *)
                        git rev-parse --verify "$1^0"
                        ;;
                esac
        }

Thanks.  Yeah, that's the obvious "hack solution": special handling
for refspecs of the kind :/text.

By the way, that is not a "hack", but is merely "working within the
given constraints".  Welcome to the real life.

Unlike "rebase" that has the luxury of possibly update the
underlying plumbing, third-party tools have to work within the
constraints of existing versions of Git.  It is better to learn that
you sometimes need to do so, and how to do so when you need to.