Lee Carver <Lee.Carver@xxxxxxxxxxxxxx> writes: > It is using the same ${tempdir} working directory that git rev-list uses > below for the ../revs file Ah, I missed that; then that should be safe. The patch looks sane. Can we have your sign-off, too, please? > > It's normally .git-rewrite/t, following the normal working directory setup > near line 205. > >> >>> >>> case "$filter_subdir" in >>> "") >>> @@ -268,7 +268,7 @@ case "$filter_subdir" in >>> esac >>> >>> git rev-list --reverse --topo-order --default HEAD \ >>> - --parents --simplify-merges $rev_args "$@" > ../revs || >>> + --parents --simplify-merges --stdin "$@" < ../parse > ../revs || >>> die "Could not get the commits" >>> commits=$(wc -l <../revs | tr -d " ") -- 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