'git-filter-branch branch' could fail producing the error: "Which ref do you want to rewrite?" if existed another branch or tag, which name was 'branch-something' or 'something/branch'. [jc: original report and fix were done between Dmitry Potapov and Dscho; I rewrote it using "rev-parse --symbolic-full-name"] Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> --- Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: >> But at that point I would say that exposing the refname dwimming >> logic to the scripts could be a much cleaner solution. > > I considered that when ripping the script from cogito, but it seemed to me > at that time that not requiring an up-to-date git for testing the script > would be better. > > Now is a different situation, however, so I agree. It was already tied to the specific git version when git-filter-branch became part of git.git ;-) I do not use filter-branch myself very often, but I think this is worth fixing. The additional --no-flags and sed are to deal with something like: --topo-order master..next although I do not offhand know if filter-branch would work with things like --topo-order and --first-parent. git-filter-branch.sh | 22 +++------------------- 1 files changed, 3 insertions(+), 19 deletions(-) diff --git a/git-filter-branch.sh b/git-filter-branch.sh index ae29f47..ebf05ca 100755 --- a/git-filter-branch.sh +++ b/git-filter-branch.sh @@ -209,25 +209,9 @@ ORIG_GIT_INDEX_FILE="$GIT_INDEX_FILE" GIT_WORK_TREE=. export GIT_DIR GIT_WORK_TREE -# These refs should be updated if their heads were rewritten - -git rev-parse --revs-only --symbolic "$@" | -while read ref -do - # normalize ref - case "$ref" in - HEAD) - ref="$(git symbolic-ref "$ref")" - ;; - refs/*) - ;; - *) - ref="$(git for-each-ref --format='%(refname)' | - grep /"$ref")" - esac - - git check-ref-format "$ref" && echo "$ref" -done > "$tempdir"/heads +# The refs should be updated if their heads were rewritten +git rev-parse --no-flags --revs-only --symbolic-full-name "$@" | +sed -e '/^^/d' >"$tempdir"/heads test -s "$tempdir"/heads || die "Which ref do you want to rewrite?" -- 1.5.4.rc2.38.gd6da3 - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
