2007-06-03 04:31:28 +04:00
|
|
|
#!/bin/sh
|
|
|
|
|
2008-09-03 12:59:31 +04:00
|
|
|
test_description='git filter-branch'
|
2007-06-03 04:31:28 +04:00
|
|
|
. ./test-lib.sh
|
filter-branch: remove multi-line headers in msg filter
df062010 (filter-branch: avoid passing commit message through sed)
introduced a regression when filtering commits with multi-line headers,
if the header contains a blank line. An example of this is a gpg-signed
commit:
$ git cat-file commit signed-commit
tree 3d4038e029712da9fc59a72afbfcc90418451630
parent 110eac945dc1713b27bdf49e74e5805db66971f0
author A U Thor <author@example.com> 1112912413 -0700
committer C O Mitter <committer@example.com> 1112912413 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iEYEABECAAYFAlYXADwACgkQE7b1Hs3eQw23CACgldB/InRyDgQwyiFyMMm3zFpj
pUsAnA+f3aMUsd9mNroloSmlOgL6jIMO
=0Hgm
-----END PGP SIGNATURE-----
Adding gpg
As a consequence, "filter-branch --msg-filter cat" (which should leave the
commit message unchanged) spills the signature (after the internal blank
line) into the original commit message.
The reason is that although the signature is indented, making the line a
whitespace only line, the "read" call is splitting the line based on
the shell's IFS, which defaults to <space><tab><newline>. The leading
space is consumed and $header_line is empty, causing the "skip header
lines" loop to exit.
The rest of the commit object is then re-used as the rewritten commit
message, causing the new message to include the signature of the
original commit.
Set IFS to an empty string for the "read" call, thus disabling the word
splitting, which causes $header_line to be set to the non-empty value ' '.
This allows the loop to fully consume the header lines before
emitting the original, intact commit message.
[jc: this is literally based on MJG's suggestion]
Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
Signed-off-by: James McCoy <vega.james@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-10-09 03:21:13 +03:00
|
|
|
. "$TEST_DIRECTORY/lib-gpg.sh"
|
2007-06-03 04:31:28 +04:00
|
|
|
|
|
|
|
test_expect_success 'setup' '
|
2010-09-11 02:06:36 +04:00
|
|
|
test_commit A &&
|
2012-07-13 00:51:37 +04:00
|
|
|
GIT_COMMITTER_DATE="@0 +0000" GIT_AUTHOR_DATE="@0 +0000" &&
|
|
|
|
test_commit --notick B &&
|
2010-09-11 02:06:36 +04:00
|
|
|
git checkout -b branch B &&
|
|
|
|
test_commit D &&
|
|
|
|
mkdir dir &&
|
|
|
|
test_commit dir/D &&
|
|
|
|
test_commit E &&
|
|
|
|
git checkout master &&
|
|
|
|
test_commit C &&
|
|
|
|
git checkout branch &&
|
|
|
|
git merge C &&
|
|
|
|
git tag F &&
|
|
|
|
test_commit G &&
|
|
|
|
test_commit H
|
2007-06-03 04:31:28 +04:00
|
|
|
'
|
2010-09-11 02:06:36 +04:00
|
|
|
# * (HEAD, branch) H
|
|
|
|
# * G
|
|
|
|
# * Merge commit 'C' into branch
|
|
|
|
# |\
|
|
|
|
# | * (master) C
|
|
|
|
# * | E
|
|
|
|
# * | dir/D
|
|
|
|
# * | D
|
|
|
|
# |/
|
|
|
|
# * B
|
|
|
|
# * A
|
|
|
|
|
2007-06-03 04:31:28 +04:00
|
|
|
|
2007-07-03 09:52:14 +04:00
|
|
|
H=$(git rev-parse H)
|
2007-06-03 04:31:28 +04:00
|
|
|
|
|
|
|
test_expect_success 'rewrite identically' '
|
2008-09-03 12:59:31 +04:00
|
|
|
git filter-branch branch
|
2007-06-03 04:31:28 +04:00
|
|
|
'
|
|
|
|
test_expect_success 'result is really identical' '
|
2007-07-23 21:34:13 +04:00
|
|
|
test $H = $(git rev-parse HEAD)
|
2007-06-03 04:31:28 +04:00
|
|
|
'
|
|
|
|
|
2008-07-24 02:15:36 +04:00
|
|
|
test_expect_success 'rewrite bare repository identically' '
|
2009-02-03 21:27:03 +03:00
|
|
|
(git config core.bare true && cd .git &&
|
|
|
|
git filter-branch branch > filter-output 2>&1 &&
|
|
|
|
! fgrep fatal filter-output)
|
2008-07-24 02:15:36 +04:00
|
|
|
'
|
|
|
|
git config core.bare false
|
|
|
|
test_expect_success 'result is really identical' '
|
|
|
|
test $H = $(git rev-parse HEAD)
|
|
|
|
'
|
|
|
|
|
2009-02-18 11:35:36 +03:00
|
|
|
TRASHDIR=$(pwd)
|
|
|
|
test_expect_success 'correct GIT_DIR while using -d' '
|
|
|
|
mkdir drepo &&
|
|
|
|
( cd drepo &&
|
|
|
|
git init &&
|
|
|
|
test_commit drepo &&
|
|
|
|
git filter-branch -d "$TRASHDIR/dfoo" \
|
|
|
|
--index-filter "cp \"$TRASHDIR\"/dfoo/backup-refs \"$TRASHDIR\"" \
|
|
|
|
) &&
|
|
|
|
grep drepo "$TRASHDIR/backup-refs"
|
|
|
|
'
|
|
|
|
|
2013-04-02 18:22:19 +04:00
|
|
|
test_expect_success 'tree-filter works with -d' '
|
|
|
|
git init drepo-tree &&
|
|
|
|
(
|
|
|
|
cd drepo-tree &&
|
|
|
|
test_commit one &&
|
|
|
|
git filter-branch -d "$TRASHDIR/dfoo" \
|
|
|
|
--tree-filter "echo changed >one.t" &&
|
|
|
|
echo changed >expect &&
|
|
|
|
git cat-file blob HEAD:one.t >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_cmp one.t actual
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2009-02-12 00:10:41 +03:00
|
|
|
test_expect_success 'Fail if commit filter fails' '
|
|
|
|
test_must_fail git filter-branch -f --commit-filter "exit 1" HEAD
|
|
|
|
'
|
|
|
|
|
2007-06-03 04:31:28 +04:00
|
|
|
test_expect_success 'rewrite, renaming a specific file' '
|
2010-09-11 02:06:36 +04:00
|
|
|
git filter-branch -f --tree-filter "mv D.t doh || :" HEAD
|
2007-06-03 04:31:28 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'test that the file was renamed' '
|
2010-09-11 02:06:36 +04:00
|
|
|
test D = "$(git show HEAD:doh --)" &&
|
|
|
|
! test -f D.t &&
|
2007-10-17 06:23:10 +04:00
|
|
|
test -f doh &&
|
2010-09-11 02:06:36 +04:00
|
|
|
test D = "$(cat doh)"
|
2008-03-31 11:14:14 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'rewrite, renaming a specific directory' '
|
2008-09-03 12:59:31 +04:00
|
|
|
git filter-branch -f --tree-filter "mv dir diroh || :" HEAD
|
2008-03-31 11:14:14 +04:00
|
|
|
'
|
|
|
|
|
2008-03-31 11:14:15 +04:00
|
|
|
test_expect_success 'test that the directory was renamed' '
|
2010-09-11 02:06:36 +04:00
|
|
|
test dir/D = "$(git show HEAD:diroh/D.t --)" &&
|
2008-03-31 11:14:14 +04:00
|
|
|
! test -d dir &&
|
|
|
|
test -d diroh &&
|
|
|
|
! test -d diroh/dir &&
|
2010-09-11 02:06:36 +04:00
|
|
|
test -f diroh/D.t &&
|
|
|
|
test dir/D = "$(cat diroh/D.t)"
|
2007-06-03 04:31:28 +04:00
|
|
|
'
|
|
|
|
|
2007-07-23 21:34:13 +04:00
|
|
|
git tag oldD HEAD~4
|
2007-06-05 19:58:13 +04:00
|
|
|
test_expect_success 'rewrite one branch, keeping a side branch' '
|
2007-07-23 21:34:13 +04:00
|
|
|
git branch modD oldD &&
|
2010-09-11 02:06:36 +04:00
|
|
|
git filter-branch -f --tree-filter "mv B.t boh || :" D..modD
|
2007-06-05 19:58:13 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'common ancestor is still common (unchanged)' '
|
2007-07-03 09:52:14 +04:00
|
|
|
test "$(git merge-base modD D)" = "$(git rev-parse B)"
|
2007-06-05 19:58:13 +04:00
|
|
|
'
|
|
|
|
|
2007-06-08 04:30:35 +04:00
|
|
|
test_expect_success 'filter subdirectory only' '
|
|
|
|
mkdir subdir &&
|
|
|
|
touch subdir/new &&
|
|
|
|
git add subdir/new &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "subdir" &&
|
2010-09-11 02:06:36 +04:00
|
|
|
echo H > A.t &&
|
2007-06-08 04:30:35 +04:00
|
|
|
test_tick &&
|
2010-09-11 02:06:36 +04:00
|
|
|
git commit -m "not subdir" A.t &&
|
2007-06-08 04:30:35 +04:00
|
|
|
echo A > subdir/new &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "again subdir" subdir/new &&
|
2010-09-11 02:06:36 +04:00
|
|
|
git rm A.t &&
|
2007-06-08 04:30:35 +04:00
|
|
|
test_tick &&
|
|
|
|
git commit -m "again not subdir" &&
|
2007-07-23 21:34:13 +04:00
|
|
|
git branch sub &&
|
2008-08-12 12:45:57 +04:00
|
|
|
git branch sub-earlier HEAD~2 &&
|
2008-09-04 03:08:23 +04:00
|
|
|
git filter-branch -f --subdirectory-filter subdir \
|
2008-08-12 12:45:57 +04:00
|
|
|
refs/heads/sub refs/heads/sub-earlier
|
2007-06-08 04:30:35 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'subdirectory filter result looks okay' '
|
2007-07-03 09:52:14 +04:00
|
|
|
test 2 = $(git rev-list sub | wc -l) &&
|
2007-06-08 04:30:35 +04:00
|
|
|
git show sub:new &&
|
2008-08-12 12:45:57 +04:00
|
|
|
test_must_fail git show sub:subdir &&
|
|
|
|
git show sub-earlier:new &&
|
|
|
|
test_must_fail git show sub-earlier:subdir
|
2007-06-08 04:30:35 +04:00
|
|
|
'
|
|
|
|
|
2008-05-26 23:09:18 +04:00
|
|
|
test_expect_success 'more setup' '
|
2007-06-09 01:28:50 +04:00
|
|
|
git checkout master &&
|
|
|
|
mkdir subdir &&
|
|
|
|
echo A > subdir/new &&
|
|
|
|
git add subdir/new &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "subdir on master" subdir/new &&
|
2010-09-11 02:06:36 +04:00
|
|
|
git rm A.t &&
|
2007-06-09 01:28:50 +04:00
|
|
|
test_tick &&
|
|
|
|
git commit -m "again subdir on master" &&
|
2008-05-26 23:09:18 +04:00
|
|
|
git merge branch
|
2007-06-09 01:28:50 +04:00
|
|
|
'
|
|
|
|
|
2007-06-21 21:05:30 +04:00
|
|
|
test_expect_success 'use index-filter to move into a subdirectory' '
|
2007-07-23 21:34:13 +04:00
|
|
|
git branch directorymoved &&
|
2008-09-03 12:59:31 +04:00
|
|
|
git filter-branch -f --index-filter \
|
2010-08-13 00:55:54 +04:00
|
|
|
"git ls-files -s | sed \"s- -&newsubdir/-\" |
|
2007-06-21 21:05:30 +04:00
|
|
|
GIT_INDEX_FILE=\$GIT_INDEX_FILE.new \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-index --index-info &&
|
2008-05-04 09:37:59 +04:00
|
|
|
mv \"\$GIT_INDEX_FILE.new\" \"\$GIT_INDEX_FILE\"" directorymoved &&
|
2010-07-29 19:10:22 +04:00
|
|
|
git diff --exit-code HEAD directorymoved:newsubdir
|
|
|
|
'
|
2007-06-21 21:05:30 +04:00
|
|
|
|
filter-branch: fail gracefully when a filter fails
A common mistake is to provide a filter which fails unwantedly. For
example, this will stop in the middle:
git filter-branch --env-filter '
test $GIT_COMMITTER_EMAIL = xyz &&
export GIT_COMMITTER_EMAIL = abc' rewritten
When $GIT_COMMITTER_EMAIL is not "xyz", the test fails, and consequently
the whole filter has a non-zero exit status. However, as demonstrated
in this example, filter-branch would just stop, and the user would be
none the wiser.
Also, a failing msg-filter would not have been caught, as was the
case with one of the tests.
This patch fixes both issues, by paying attention to the exit status
of msg-filter, and by saying what failed before exiting.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-04 18:36:01 +04:00
|
|
|
test_expect_success 'stops when msg filter fails' '
|
2007-07-23 21:34:13 +04:00
|
|
|
old=$(git rev-parse HEAD) &&
|
2008-09-03 12:59:31 +04:00
|
|
|
test_must_fail git filter-branch -f --msg-filter false HEAD &&
|
2007-07-23 21:34:13 +04:00
|
|
|
test $old = $(git rev-parse HEAD) &&
|
|
|
|
rm -rf .git-rewrite
|
filter-branch: fail gracefully when a filter fails
A common mistake is to provide a filter which fails unwantedly. For
example, this will stop in the middle:
git filter-branch --env-filter '
test $GIT_COMMITTER_EMAIL = xyz &&
export GIT_COMMITTER_EMAIL = abc' rewritten
When $GIT_COMMITTER_EMAIL is not "xyz", the test fails, and consequently
the whole filter has a non-zero exit status. However, as demonstrated
in this example, filter-branch would just stop, and the user would be
none the wiser.
Also, a failing msg-filter would not have been caught, as was the
case with one of the tests.
This patch fixes both issues, by paying attention to the exit status
of msg-filter, and by saying what failed before exiting.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-04 18:36:01 +04:00
|
|
|
'
|
|
|
|
|
2007-07-03 20:50:19 +04:00
|
|
|
test_expect_success 'author information is preserved' '
|
|
|
|
: > i &&
|
|
|
|
git add i &&
|
|
|
|
test_tick &&
|
|
|
|
GIT_AUTHOR_NAME="B V Uips" git commit -m bvuips &&
|
2007-07-23 21:34:13 +04:00
|
|
|
git branch preserved-author &&
|
2012-10-18 14:33:02 +04:00
|
|
|
(sane_unset GIT_AUTHOR_NAME &&
|
|
|
|
git filter-branch -f --msg-filter "cat; \
|
filter-branch: fail gracefully when a filter fails
A common mistake is to provide a filter which fails unwantedly. For
example, this will stop in the middle:
git filter-branch --env-filter '
test $GIT_COMMITTER_EMAIL = xyz &&
export GIT_COMMITTER_EMAIL = abc' rewritten
When $GIT_COMMITTER_EMAIL is not "xyz", the test fails, and consequently
the whole filter has a non-zero exit status. However, as demonstrated
in this example, filter-branch would just stop, and the user would be
none the wiser.
Also, a failing msg-filter would not have been caught, as was the
case with one of the tests.
This patch fixes both issues, by paying attention to the exit status
of msg-filter, and by saying what failed before exiting.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-04 18:36:01 +04:00
|
|
|
test \$GIT_COMMIT != $(git rev-parse master) || \
|
2007-07-03 20:50:19 +04:00
|
|
|
echo Hallo" \
|
2012-10-18 14:33:02 +04:00
|
|
|
preserved-author) &&
|
2007-07-03 20:50:19 +04:00
|
|
|
test 1 = $(git rev-list --author="B V Uips" preserved-author | wc -l)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success "remove a certain author's commits" '
|
|
|
|
echo i > i &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m i i &&
|
2007-07-23 21:34:13 +04:00
|
|
|
git branch removed-author &&
|
2008-09-03 12:59:31 +04:00
|
|
|
git filter-branch -f --commit-filter "\
|
2007-07-03 20:50:19 +04:00
|
|
|
if [ \"\$GIT_AUTHOR_NAME\" = \"B V Uips\" ];\
|
|
|
|
then\
|
2007-08-31 23:06:27 +04:00
|
|
|
skip_commit \"\$@\";
|
2007-07-03 20:50:19 +04:00
|
|
|
else\
|
|
|
|
git commit-tree \"\$@\";\
|
|
|
|
fi" removed-author &&
|
|
|
|
cnt1=$(git rev-list master | wc -l) &&
|
|
|
|
cnt2=$(git rev-list removed-author | wc -l) &&
|
|
|
|
test $cnt1 -eq $(($cnt2 + 1)) &&
|
|
|
|
test 0 = $(git rev-list --author="B V Uips" removed-author | wc -l)
|
|
|
|
'
|
|
|
|
|
2007-07-23 21:34:13 +04:00
|
|
|
test_expect_success 'barf on invalid name' '
|
2008-03-20 19:54:30 +03:00
|
|
|
test_must_fail git filter-branch -f master xy-problem &&
|
|
|
|
test_must_fail git filter-branch -f HEAD^
|
2007-07-23 21:34:13 +04:00
|
|
|
'
|
|
|
|
|
2007-08-31 23:05:36 +04:00
|
|
|
test_expect_success '"map" works in commit filter' '
|
|
|
|
git filter-branch -f --commit-filter "\
|
|
|
|
parent=\$(git rev-parse \$GIT_COMMIT^) &&
|
|
|
|
mapped=\$(map \$parent) &&
|
|
|
|
actual=\$(echo \"\$@\" | sed \"s/^.*-p //\") &&
|
|
|
|
test \$mapped = \$actual &&
|
|
|
|
git commit-tree \"\$@\";" master~2..master &&
|
|
|
|
git rev-parse --verify master
|
|
|
|
'
|
|
|
|
|
2008-02-16 10:57:26 +03:00
|
|
|
test_expect_success 'Name needing quotes' '
|
|
|
|
|
|
|
|
git checkout -b rerere A &&
|
|
|
|
mkdir foo &&
|
|
|
|
name="れれれ" &&
|
|
|
|
>foo/$name &&
|
|
|
|
git add foo &&
|
|
|
|
git commit -m "Adding a file" &&
|
|
|
|
git filter-branch --tree-filter "rm -fr foo" &&
|
2008-03-20 19:54:30 +03:00
|
|
|
test_must_fail git ls-files --error-unmatch "foo/$name" &&
|
2008-02-16 10:57:26 +03:00
|
|
|
test $(git rev-parse --verify rerere) != $(git rev-parse --verify A)
|
|
|
|
|
|
|
|
'
|
|
|
|
|
2008-03-08 23:25:58 +03:00
|
|
|
test_expect_success 'Subdirectory filter with disappearing trees' '
|
|
|
|
git reset --hard &&
|
|
|
|
git checkout master &&
|
|
|
|
|
|
|
|
mkdir foo &&
|
|
|
|
touch foo/bar &&
|
|
|
|
git add foo &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "Adding foo" &&
|
|
|
|
|
|
|
|
git rm -r foo &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "Removing foo" &&
|
|
|
|
|
|
|
|
mkdir foo &&
|
|
|
|
touch foo/bar &&
|
|
|
|
git add foo &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -m "Re-adding foo" &&
|
|
|
|
|
|
|
|
git filter-branch -f --subdirectory-filter foo &&
|
|
|
|
test $(git rev-list master | wc -l) = 3
|
|
|
|
'
|
|
|
|
|
2008-03-26 18:47:09 +03:00
|
|
|
test_expect_success 'Tag name filtering retains tag message' '
|
|
|
|
git tag -m atag T &&
|
|
|
|
git cat-file tag T > expect &&
|
|
|
|
git filter-branch -f --tag-name-filter cat &&
|
|
|
|
git cat-file tag T > actual &&
|
2008-05-24 09:28:56 +04:00
|
|
|
test_cmp expect actual
|
2008-03-26 18:47:09 +03:00
|
|
|
'
|
|
|
|
|
|
|
|
faux_gpg_tag='object XXXXXX
|
|
|
|
type commit
|
|
|
|
tag S
|
|
|
|
tagger T A Gger <tagger@example.com> 1206026339 -0500
|
|
|
|
|
|
|
|
This is a faux gpg signed tag.
|
|
|
|
-----BEGIN PGP SIGNATURE-----
|
|
|
|
Version: FauxGPG v0.0.0 (FAUX/Linux)
|
|
|
|
|
|
|
|
gdsfoewhxu/6l06f1kxyxhKdZkrcbaiOMtkJUA9ITAc1mlamh0ooasxkH1XwMbYQ
|
|
|
|
acmwXaWET20H0GeAGP+7vow=
|
|
|
|
=agpO
|
|
|
|
-----END PGP SIGNATURE-----
|
|
|
|
'
|
|
|
|
test_expect_success 'Tag name filtering strips gpg signature' '
|
|
|
|
sha1=$(git rev-parse HEAD) &&
|
|
|
|
sha1t=$(echo "$faux_gpg_tag" | sed -e s/XXXXXX/$sha1/ | git mktag) &&
|
|
|
|
git update-ref "refs/tags/S" "$sha1t" &&
|
|
|
|
echo "$faux_gpg_tag" | sed -e s/XXXXXX/$sha1/ | head -n 6 > expect &&
|
|
|
|
git filter-branch -f --tag-name-filter cat &&
|
|
|
|
git cat-file tag S > actual &&
|
2008-05-24 09:28:56 +04:00
|
|
|
test_cmp expect actual
|
2008-03-26 18:47:09 +03:00
|
|
|
'
|
|
|
|
|
filter-branch: remove multi-line headers in msg filter
df062010 (filter-branch: avoid passing commit message through sed)
introduced a regression when filtering commits with multi-line headers,
if the header contains a blank line. An example of this is a gpg-signed
commit:
$ git cat-file commit signed-commit
tree 3d4038e029712da9fc59a72afbfcc90418451630
parent 110eac945dc1713b27bdf49e74e5805db66971f0
author A U Thor <author@example.com> 1112912413 -0700
committer C O Mitter <committer@example.com> 1112912413 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iEYEABECAAYFAlYXADwACgkQE7b1Hs3eQw23CACgldB/InRyDgQwyiFyMMm3zFpj
pUsAnA+f3aMUsd9mNroloSmlOgL6jIMO
=0Hgm
-----END PGP SIGNATURE-----
Adding gpg
As a consequence, "filter-branch --msg-filter cat" (which should leave the
commit message unchanged) spills the signature (after the internal blank
line) into the original commit message.
The reason is that although the signature is indented, making the line a
whitespace only line, the "read" call is splitting the line based on
the shell's IFS, which defaults to <space><tab><newline>. The leading
space is consumed and $header_line is empty, causing the "skip header
lines" loop to exit.
The rest of the commit object is then re-used as the rewritten commit
message, causing the new message to include the signature of the
original commit.
Set IFS to an empty string for the "read" call, thus disabling the word
splitting, which causes $header_line to be set to the non-empty value ' '.
This allows the loop to fully consume the header lines before
emitting the original, intact commit message.
[jc: this is literally based on MJG's suggestion]
Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
Signed-off-by: James McCoy <vega.james@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-10-09 03:21:13 +03:00
|
|
|
test_expect_success GPG 'Filtering retains message of gpg signed commit' '
|
|
|
|
mkdir gpg &&
|
|
|
|
touch gpg/foo &&
|
|
|
|
git add gpg &&
|
|
|
|
test_tick &&
|
|
|
|
git commit -S -m "Adding gpg" &&
|
|
|
|
|
|
|
|
git log -1 --format="%s" > expect &&
|
|
|
|
git filter-branch -f --msg-filter "cat" &&
|
|
|
|
git log -1 --format="%s" > actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2008-08-21 18:45:11 +04:00
|
|
|
test_expect_success 'Tag name filtering allows slashes in tag names' '
|
|
|
|
git tag -m tag-with-slash X/1 &&
|
|
|
|
git cat-file tag X/1 | sed -e s,X/1,X/2, > expect &&
|
|
|
|
git filter-branch -f --tag-name-filter "echo X/2" &&
|
|
|
|
git cat-file tag X/2 > actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
2017-02-23 11:27:33 +03:00
|
|
|
test_expect_success 'setup --prune-empty comparisons' '
|
|
|
|
git checkout --orphan master-no-a &&
|
|
|
|
git rm -rf . &&
|
|
|
|
unset test_tick &&
|
|
|
|
test_tick &&
|
|
|
|
GIT_COMMITTER_DATE="@0 +0000" GIT_AUTHOR_DATE="@0 +0000" &&
|
|
|
|
test_commit --notick B B.t B Bx &&
|
|
|
|
git checkout -b branch-no-a Bx &&
|
|
|
|
test_commit D D.t D Dx &&
|
|
|
|
mkdir dir &&
|
|
|
|
test_commit dir/D dir/D.t dir/D dir/Dx &&
|
|
|
|
test_commit E E.t E Ex &&
|
|
|
|
git checkout master-no-a &&
|
|
|
|
test_commit C C.t C Cx &&
|
|
|
|
git checkout branch-no-a &&
|
|
|
|
git merge Cx -m "Merge tag '\''C'\'' into branch" &&
|
|
|
|
git tag Fx &&
|
|
|
|
test_commit G G.t G Gx &&
|
|
|
|
test_commit H H.t H Hx &&
|
|
|
|
git checkout branch
|
|
|
|
'
|
2008-08-21 18:45:11 +04:00
|
|
|
|
2008-10-31 12:12:21 +03:00
|
|
|
test_expect_success 'Prune empty commits' '
|
|
|
|
git rev-list HEAD > expect &&
|
2010-09-11 02:06:36 +04:00
|
|
|
test_commit to_remove &&
|
|
|
|
git filter-branch -f --index-filter "git update-index --remove to_remove.t" --prune-empty HEAD &&
|
2008-10-31 12:12:21 +03:00
|
|
|
git rev-list HEAD > actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2014-07-01 01:20:27 +04:00
|
|
|
test_expect_success 'prune empty collapsed merges' '
|
|
|
|
test_config merge.ff false &&
|
|
|
|
git rev-list HEAD >expect &&
|
|
|
|
test_commit to_remove_2 &&
|
|
|
|
git reset --hard HEAD^ &&
|
|
|
|
test_merge non-ff to_remove_2 &&
|
|
|
|
git filter-branch -f --index-filter "git update-index --remove to_remove_2.t" --prune-empty HEAD &&
|
|
|
|
git rev-list HEAD >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
filter-branch: resolve $commit^{tree} in no-index case
Commit 348d4f2 (filter-branch: skip index read/write when
possible, 2015-11-06) taught filter-branch to optimize out
the final "git write-tree" when we know we haven't touched
the tree with any of our filters. It does by simply putting
the literal text "$commit^{tree}" into the "$tree" variable,
avoiding a useless rev-parse call.
However, when we pass this to git_commit_non_empty_tree(),
it gets confused; it resolves "$commit^{tree}" itself, and
compares our string to the 40-hex sha1, which obviously
doesn't match. As a result, "--prune-empty" (or any custom
filter using git_commit_non_empty_tree) will fail to drop
an empty commit (when filter-branch is used without a tree
or index filter).
Let's resolve $tree to the 40-hex ourselves, so that
git_commit_non_empty_tree can work. Unfortunately, this is a
bit slower due to the extra process overhead:
$ cd t/perf && ./run 348d4f2 HEAD p7000-filter-branch.sh
[...]
Test 348d4f2 HEAD
--------------------------------------------------------------
7000.2: noop filter 3.76(0.24+0.26) 4.54(0.28+0.24) +20.7%
We could try to make git_commit_non_empty_tree more clever.
However, the value of $tree here is technically
user-visible. The user can provide arbitrary shell code at
this stage, which could itself have a similar assumption to
what is in git_commit_non_empty_tree. So the conservative
choice to fix this regression is to take the 20% hit and
give the pre-348d4f2 behavior. We still end up much faster
than before the optimization:
$ cd t/perf && ./run 348d4f2^ HEAD p7000-filter-branch.sh
[...]
Test 348d4f2^ HEAD
--------------------------------------------------------------
7000.2: noop filter 9.51(4.32+0.40) 4.51(0.28+0.23) -52.6%
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-01-20 01:07:22 +03:00
|
|
|
test_expect_success 'prune empty works even without index/tree filters' '
|
|
|
|
git rev-list HEAD >expect &&
|
|
|
|
git commit --allow-empty -m empty &&
|
|
|
|
git filter-branch -f --prune-empty HEAD &&
|
|
|
|
git rev-list HEAD >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2017-02-23 11:27:35 +03:00
|
|
|
test_expect_success '--prune-empty is able to prune root commit' '
|
2017-02-23 11:27:33 +03:00
|
|
|
git rev-list branch-no-a >expect &&
|
|
|
|
git branch testing H &&
|
|
|
|
git filter-branch -f --prune-empty --index-filter "git update-index --remove A.t" testing &&
|
|
|
|
git rev-list testing >actual &&
|
|
|
|
git branch -D testing &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2017-02-23 11:27:35 +03:00
|
|
|
test_expect_success '--prune-empty is able to prune entire branch' '
|
2017-02-23 11:27:34 +03:00
|
|
|
git branch prune-entire B &&
|
|
|
|
git filter-branch -f --prune-empty --index-filter "git update-index --remove A.t B.t" prune-entire &&
|
|
|
|
test_path_is_missing .git/refs/heads/prune-entire &&
|
|
|
|
test_must_fail git reflog exists refs/heads/prune-entire
|
|
|
|
'
|
|
|
|
|
2009-11-11 00:04:51 +03:00
|
|
|
test_expect_success '--remap-to-ancestor with filename filters' '
|
|
|
|
git checkout master &&
|
|
|
|
git reset --hard A &&
|
|
|
|
test_commit add-foo foo 1 &&
|
|
|
|
git branch moved-foo &&
|
|
|
|
test_commit add-bar bar a &&
|
|
|
|
git branch invariant &&
|
|
|
|
orig_invariant=$(git rev-parse invariant) &&
|
|
|
|
git branch moved-bar &&
|
|
|
|
test_commit change-foo foo 2 &&
|
|
|
|
git filter-branch -f --remap-to-ancestor \
|
|
|
|
moved-foo moved-bar A..master \
|
|
|
|
-- -- foo &&
|
|
|
|
test $(git rev-parse moved-foo) = $(git rev-parse moved-bar) &&
|
|
|
|
test $(git rev-parse moved-foo) = $(git rev-parse master^) &&
|
|
|
|
test $orig_invariant = $(git rev-parse invariant)
|
|
|
|
'
|
|
|
|
|
2010-08-28 00:44:56 +04:00
|
|
|
test_expect_success 'automatic remapping to ancestor with filename filters' '
|
|
|
|
git checkout master &&
|
|
|
|
git reset --hard A &&
|
|
|
|
test_commit add-foo2 foo 1 &&
|
|
|
|
git branch moved-foo2 &&
|
|
|
|
test_commit add-bar2 bar a &&
|
|
|
|
git branch invariant2 &&
|
|
|
|
orig_invariant=$(git rev-parse invariant2) &&
|
|
|
|
git branch moved-bar2 &&
|
|
|
|
test_commit change-foo2 foo 2 &&
|
|
|
|
git filter-branch -f \
|
|
|
|
moved-foo2 moved-bar2 A..master \
|
|
|
|
-- -- foo &&
|
|
|
|
test $(git rev-parse moved-foo2) = $(git rev-parse moved-bar2) &&
|
|
|
|
test $(git rev-parse moved-foo2) = $(git rev-parse master^) &&
|
|
|
|
test $orig_invariant = $(git rev-parse invariant2)
|
|
|
|
'
|
|
|
|
|
2010-01-28 12:08:47 +03:00
|
|
|
test_expect_success 'setup submodule' '
|
|
|
|
rm -fr ?* .git &&
|
|
|
|
git init &&
|
|
|
|
test_commit file &&
|
|
|
|
mkdir submod &&
|
|
|
|
submodurl="$PWD/submod" &&
|
|
|
|
( cd submod &&
|
|
|
|
git init &&
|
|
|
|
test_commit file-in-submod ) &&
|
|
|
|
git submodule add "$submodurl" &&
|
|
|
|
git commit -m "added submodule" &&
|
|
|
|
test_commit add-file &&
|
|
|
|
( cd submod && test_commit add-in-submodule ) &&
|
|
|
|
git add submod &&
|
|
|
|
git commit -m "changed submodule" &&
|
|
|
|
git branch original HEAD
|
|
|
|
'
|
|
|
|
|
2016-01-07 16:51:48 +03:00
|
|
|
orig_head=$(git show-ref --hash --head HEAD)
|
2010-01-28 12:08:47 +03:00
|
|
|
|
|
|
|
test_expect_success 'rewrite submodule with another content' '
|
|
|
|
git filter-branch --tree-filter "test -d submod && {
|
|
|
|
rm -rf submod &&
|
|
|
|
git rm -rf --quiet submod &&
|
|
|
|
mkdir submod &&
|
|
|
|
: > submod/file
|
|
|
|
} || :" HEAD &&
|
2016-01-07 16:51:48 +03:00
|
|
|
test $orig_head != $(git show-ref --hash --head HEAD)
|
2010-01-28 12:08:47 +03:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'replace submodule revision' '
|
|
|
|
git reset --hard original &&
|
|
|
|
git filter-branch -f --tree-filter \
|
|
|
|
"if git ls-files --error-unmatch -- submod > /dev/null 2>&1
|
|
|
|
then git update-index --cacheinfo 160000 0123456789012345678901234567890123456789 submod
|
|
|
|
fi" HEAD &&
|
2016-01-07 16:51:48 +03:00
|
|
|
test $orig_head != $(git show-ref --hash --head HEAD)
|
2010-01-28 12:08:47 +03:00
|
|
|
'
|
|
|
|
|
2015-04-29 18:48:58 +03:00
|
|
|
test_expect_success 'filter commit message without trailing newline' '
|
|
|
|
git reset --hard original &&
|
|
|
|
commit=$(printf "no newline" | git commit-tree HEAD^{tree}) &&
|
|
|
|
git update-ref refs/heads/no-newline $commit &&
|
|
|
|
git filter-branch -f refs/heads/no-newline &&
|
|
|
|
echo $commit >expect &&
|
|
|
|
git rev-parse refs/heads/no-newline >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2015-11-23 15:23:16 +03:00
|
|
|
test_expect_success 'tree-filter deals with object name vs pathname ambiguity' '
|
|
|
|
test_when_finished "git reset --hard original" &&
|
|
|
|
ambiguous=$(git rev-list -1 HEAD) &&
|
|
|
|
git filter-branch --tree-filter "mv file.t $ambiguous" HEAD^.. &&
|
|
|
|
git show HEAD:$ambiguous
|
|
|
|
'
|
|
|
|
|
2007-06-03 04:31:28 +04:00
|
|
|
test_done
|