{"thread":{"id":"13621","subject":"[PATCH] Add a test for another combination of --reference","startedAt":"2008-05-22T22:03:05Z","lastAt":"2008-05-23T02:03:36Z","messageCount":3,"participants":["Daniel Barkalow","Johan Herland"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"77501","messageId":"alpine.LNX.1.00.0805221800090.19665@iabervon.org","threadId":"13621","inReplyTo":null,"subject":"[PATCH] Add a test for another combination of --reference","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-05-22T22:03:05Z","receivedAt":"2008-05-22T22:03:05Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"From: Johan Herland \n\nIn this case, the reference repository has some useful loose objects,\nbut not all useful objects, and we make sure that we can find the\nobjects we fetch from the repository we're cloning in the new\nrepository, instead of potentially being distracted by the reference\nrepository.\n\nDoing the wrong thing in a builtin-clone implementation would lead to\nthis looking for an object in the wrong place, not finding it (because\nit's only in the right place), and crashing.\n\nSigned-off-by: Johan Herland \nSigned-off-by: Daniel Barkalow \n---\n t/t5700-clone-reference.sh | 21 +++++++++++++++++++++\n 1 files changed, 21 insertions(+), 0 deletions(-)\n\ndiff --git a/t/t5700-clone-reference.sh b/t/t5700-clone-reference.sh\nindex d318780..0c42d9f 100755\n--- a/t/t5700-clone-reference.sh\n+++ b/t/t5700-clone-reference.sh\n@@ -118,4 +118,25 @@ test_expect_success 'cloning with reference being subset of source (-l -s)' \\\n \n cd \"$base_dir\"\n \n+test_expect_success 'preparing alternate repository #1' \\\n+'test_create_repo F && cd F &&\n+echo first > file1 &&\n+git add file1 &&\n+git commit -m initial'\n+\n+cd \"$base_dir\"\n+\n+test_expect_success 'cloning alternate repo #2 and adding changes to repo #1' \\\n+'git clone F G && cd F &&\n+echo second > file2 &&\n+git add file2 &&\n+git commit -m addition'\n+\n+cd \"$base_dir\"\n+\n+test_expect_success 'cloning alternate repo #1, using #2 as reference' \\\n+'git clone --reference G F H'\n+\n+cd \"$base_dir\"\n+\n test_done\n-- \n1.5.3.7\n"},{"id":"77508","messageId":"200805230034.33567.johan@herland.net","threadId":"13621","inReplyTo":"alpine.LNX.1.00.0805221800090.19665@iabervon.org","subject":"Re: [PATCH] Add a test for another combination of --reference","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-05-22T22:34:33Z","receivedAt":"2008-05-22T22:34:33Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"On Friday 23 May 2008, Daniel Barkalow wrote:\n> From: Johan Herland \n> \n> In this case, the reference repository has some useful loose objects,\n> but not all useful objects, and we make sure that we can find the\n> objects we fetch from the repository we're cloning in the new\n> repository, instead of potentially being distracted by the reference\n> repository.\n> \n> Doing the wrong thing in a builtin-clone implementation would lead to\n> this looking for an object in the wrong place, not finding it (because\n> it's only in the right place), and crashing.\n> \n> Signed-off-by: Johan Herland \n> Signed-off-by: Daniel Barkalow \n\nThanks again.\n\n...Johan\n\nPS: I'm holding off on submitting my builtin-clone-packs-refs series until\n1.5.6 is out. I think builtin-clone has seen enough action in this cycle. :)\n\n-- \nJohan Herland, \nwww.herland.net\n"},{"id":"77516","messageId":"alpine.LNX.1.00.0805222202060.19665@iabervon.org","threadId":"13621","inReplyTo":"200805230034.33567.johan@herland.net","subject":"Re: [PATCH] Add a test for another combination of --reference","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-05-23T02:03:36Z","receivedAt":"2008-05-23T02:03:36Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"On Fri, 23 May 2008, Johan Herland wrote:\n\n> On Friday 23 May 2008, Daniel Barkalow wrote:\n> > From: Johan Herland \n> > \n> > In this case, the reference repository has some useful loose objects,\n> > but not all useful objects, and we make sure that we can find the\n> > objects we fetch from the repository we're cloning in the new\n> > repository, instead of potentially being distracted by the reference\n> > repository.\n> > \n> > Doing the wrong thing in a builtin-clone implementation would lead to\n> > this looking for an object in the wrong place, not finding it (because\n> > it's only in the right place), and crashing.\n> > \n> > Signed-off-by: Johan Herland \n> > Signed-off-by: Daniel Barkalow \n> \n> Thanks again.\n> \n> ...Johan\n> \n> PS: I'm holding off on submitting my builtin-clone-packs-refs series until\n> 1.5.6 is out. I think builtin-clone has seen enough action in this cycle. :)\n\nYeah, I agree.\n\n\t-Daniel\n*This .sig left intentionally blank*\n"}]}