Re: [PATCH] checkout tests: enable test with complex relative path
On 10/09/2013 09:39 PM, Jonathan Nieder wrote: Stefan Beller wrote: This test was put in, but commented out in fed1b5ca (2007-11-09, git-checkout: Test for relative path use.) It's been a while since 2007 and the intended test case works now. (I could not find the enabling commit in ls-files however.) Bisects to v1.5.5-rc0~184^2~3 (setup: sanitize absolute and funny paths in get_pathspec(), 2008-01-28). Thanks. Do you want me to resend a reworded patch, or could you amend that information to this patch? -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] checkout tests: enable test with complex relative path
Stefan Beller wrote: On 10/09/2013 09:39 PM, Jonathan Nieder wrote: Stefan Beller wrote: This test was put in, but commented out in fed1b5ca (2007-11-09, git-checkout: Test for relative path use.) It's been a while since 2007 and the intended test case works now. (I could not find the enabling commit in ls-files however.) Bisects to v1.5.5-rc0~184^2~3 (setup: sanitize absolute and funny paths in get_pathspec(), 2008-01-28). Thanks. Do you want me to resend a reworded patch, or could you amend that information to this patch? Added locally, along with a couple of other nits. Will push out soon, and then a sanity check on the result would be much appreciated. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] checkout tests: enable test with complex relative path
This test was put in, but commented out in fed1b5ca (2007-11-09, git-checkout: Test for relative path use.) It's been a while since 2007 and the intended test case works now. (I could not find the enabling commit in ls-files however.) The code in question however did not change into the sub directory, so we still need to add a 'cd'. Also a test for the file content has been added. This is already part of the other tests for checkout. Signed-off-by: Stefan Beller stefanbel...@googlemail.com --- t/t2008-checkout-subdir.sh | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/t/t2008-checkout-subdir.sh b/t/t2008-checkout-subdir.sh index 3e098ab..399655f 100755 --- a/t/t2008-checkout-subdir.sh +++ b/t/t2008-checkout-subdir.sh @@ -58,13 +58,14 @@ test_expect_success 'checkout with simple prefix' ' ' -# This is not expected to work as ls-files was not designed -# to deal with such. Enable it when ls-files is updated. -: test_expect_success 'checkout with complex relative path' ' - - rm file1 - git checkout HEAD -- ../dir1/../dir1/file1 test -f ./file1 - +test_expect_success 'checkout with complex relative path' ' + ( + cd dir1 + rm file1 + git checkout HEAD -- ../dir1/../dir1/file1 + test -f file1 + test hello = $(cat file1) + ) ' test_expect_success 'relative path outside tree should fail' \ -- 1.8.4.1.471.g53f64e4.dirty -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html