Kristian Høgsberg <krh@xxxxxxxxxx> writes: > Signed-off-by: Kristian Høgsberg <krh@xxxxxxxxxx> > --- > t/t7501-commit.sh | 10 ++++++++++ > 1 files changed, 10 insertions(+), 0 deletions(-) > > diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh > index b151b51..3f2112a 100644 > --- a/t/t7501-commit.sh > +++ b/t/t7501-commit.sh > @@ -163,4 +163,14 @@ test_expect_success 'partial commit that involves removal (3)' ' > > ' > > +author="The Real Author <someguy@xxxxxxxxxxxxx>" > +test_expect_success 'amend commit to fix author' ' > + > + git reset --hard > + git cat-file -p HEAD | sed -e "s/author.*>/author $author/" > expected && > + git commit --amend --author="$author" && > + git cat-file -p HEAD > current && > + diff expected current > + > +' > test_done This can't be right. How are you ignoring the differences in committer dates? By the way, I _think_ git-commit.sh allows fixing author name/email without molesting the author timestamp (i.e. takes it from the amended commit). That should probably be checked with the test as well. - 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