Using \t to represent a tab character is not portable beyond GNU sed (see e.g. GNU sed's info pages). Use printf to generate the tab instead. Signed-off-by: Thomas Rast <trast@xxxxxxxxxxxxxxx> --- Adam Mercer wrote: > > What OS is this? > This was on Mac OS X 10.6. Where sed is BSD sed. That's very funny however, since it means that nobody should ever have had a successful test run on OS X with the preinstalled tools. What gives? Documentation/git-filter-branch.txt | 3 ++- t/t7003-filter-branch.sh | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/Documentation/git-filter-branch.txt b/Documentation/git-filter-branch.txt index 020028c..7e3ff8e 100644 --- a/Documentation/git-filter-branch.txt +++ b/Documentation/git-filter-branch.txt @@ -357,8 +357,9 @@ git filter-branch ... D..H --not C To move the whole tree into a subdirectory, or remove it from there: --------------------------------------------------------------- +export TAB="$(printf '\t')" git filter-branch --index-filter \ - 'git ls-files -s | sed "s-\t\"*-&newsubdir/-" | + 'git ls-files -s | sed "s-$TAB\"*-&newsubdir/-" | GIT_INDEX_FILE=$GIT_INDEX_FILE.new \ git update-index --index-info && mv $GIT_INDEX_FILE.new $GIT_INDEX_FILE' HEAD diff --git a/t/t7003-filter-branch.sh b/t/t7003-filter-branch.sh index 0da13a8..e90da6d 100755 --- a/t/t7003-filter-branch.sh +++ b/t/t7003-filter-branch.sh @@ -140,10 +140,11 @@ test_expect_success 'more setup' ' git merge branch ' +export TAB="$(printf '\t')" test_expect_success 'use index-filter to move into a subdirectory' ' git branch directorymoved && git filter-branch -f --index-filter \ - "git ls-files -s | sed \"s-\\t-&newsubdir/-\" | + "git ls-files -s | sed \"s-$TAB-&newsubdir/-\" | GIT_INDEX_FILE=\$GIT_INDEX_FILE.new \ git update-index --index-info && mv \"\$GIT_INDEX_FILE.new\" \"\$GIT_INDEX_FILE\"" directorymoved && -- 1.7.2.1.342.g676a4 -- 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