Re: [PATCH 07/11] t2100: use test_ln_s_add to remove SYMLINKS prerequisite

2013-06-05 Thread Johannes Sixt
Am 05.06.2013 00:04, schrieb Junio C Hamano: > Johannes Sixt writes: > >> @@ -62,12 +57,7 @@ test_expect_success 'git update-index to add conflicting >> file path2 should fail' >> >> test_expect_success 'git update-index to add conflicting symlink path3 >> should fail' ' >> >> -if test

Re: [PATCH 07/11] t2100: use test_ln_s_add to remove SYMLINKS prerequisite

2013-06-04 Thread Junio C Hamano
Johannes Sixt writes: > @@ -62,12 +57,7 @@ test_expect_success 'git update-index to add conflicting > file path2 should fail' > > test_expect_success 'git update-index to add conflicting symlink path3 > should fail' ' > > - if test_have_prereq SYMLINKS > - then > - ln -

[PATCH 07/11] t2100: use test_ln_s_add to remove SYMLINKS prerequisite

2013-06-01 Thread Johannes Sixt
One converted case expects 'git update-index' to fail. We cannot use 'test_must_fail test_ln_s_add ...', because that would mistakenly pass if a command other than the final git-update-index of test_ln_s_add failed. Hence, use test_ln_s to create the symbolic link and keep the explicit 'test_must_f