Signed-off-by: Peter Baumann <waste.manager@xxxxxx> --- On Wed, Apr 18, 2007 at 02:31:29PM -0700, Junio C Hamano wrote: > > Oh, I never questioned that you made that basic case work. I > was worried about not making sure the symlink we are looking at > really is the case we are willing to handle, and not erroring > out if that is not the case, perhaps like the attached patch on > top of yours. > > An additional test or two in t/t3210 would be nice to accompany > this change. > Something like this? t/t3210-pack-refs.sh | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/t/t3210-pack-refs.sh b/t/t3210-pack-refs.sh index f0c7e22..bf63954 100755 --- a/t/t3210-pack-refs.sh +++ b/t/t3210-pack-refs.sh @@ -105,4 +105,12 @@ test_expect_success 'pack, prune and repack' ' diff all-of-them again ' +test_expect_success \ + 'derefence symlinks for packed-refs' \ + 'mv -f .git/packed-refs .git/real_packed-refs && + ln -s real_packed-refs .git/packed-refs && + git-tag z && + git-pack-refs --all --prune && + diff .git/real_packed-refs .git/packed-refs' + test_done -- 1.5.1 - 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