{"thread":{"id":"31570","subject":"[PATCH] t/perf: add \"trash directory\" to .gitignore","startedAt":"2012-09-17T17:06:18Z","lastAt":"2012-09-18T20:57:50Z","messageCount":12,"participants":["Ramkumar Ramachandra","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"199251","messageId":"1347901579-5545-1-git-send-email-artagnon@gmail.com","threadId":"31570","inReplyTo":null,"subject":"[PATCH] t/perf: add \"trash directory\" to .gitignore","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-17T17:06:18Z","receivedAt":"2012-09-17T17:06:18Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Signed-off-by: Ramkumar Ramachandra \n---\n t/perf/.gitignore | 5 +++--\n 1 files changed, 3 insertions(+), 2 deletions(-)\n\ndiff --git a/t/perf/.gitignore b/t/perf/.gitignore\nindex 50f5cc1..0061cbc 100644\n--- a/t/perf/.gitignore\n+++ b/t/perf/.gitignore\n@@ -1,2 +1,3 @@\n-build/\n-test-results/\n+/build\n+/test-results\n+/trash directory*\n-- \n1.7.8.1.362.g5d6df.dirty\n"},{"id":"199252","messageId":"1347901579-5545-2-git-send-email-artagnon@gmail.com","threadId":"31570","inReplyTo":"1347901579-5545-1-git-send-email-artagnon@gmail.com","subject":"[PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-17T17:06:19Z","receivedAt":"2012-09-17T17:06:19Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"When tests were run without building git, the following error message\nwas displayed:\n\n .: 54: Can't open /path/to/git/source/t/../GIT-BUILD-OPTIONS\n\nChange this to display a more user-friendly error message:\n\n error: you do not seem to have built git yet.\n\nSigned-off-by: Ramkumar Ramachandra \n---\n t/t0000-basic.sh | 10 ----------\n t/test-lib.sh | 6 ++++++\n 2 files changed, 6 insertions(+), 10 deletions(-)\n\ndiff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\nindex ae6a3f0..08677df 100755\n--- a/t/t0000-basic.sh\n+++ b/t/t0000-basic.sh\n@@ -18,16 +18,6 @@ swapping compression and hashing order, the person who is making the\n modification *should* take notice and update the test vectors here.\n '\n \n-################################################################\n-# It appears that people try to run tests without building...\n-\n-../git >/dev/null\n-if test $? != 1\n-then\n-\techo >&2 'You do not seem to have built git yet.'\n-\texit 1\n-fi\n-\n . ./test-lib.sh\n \n ################################################################\ndiff --git a/t/test-lib.sh b/t/test-lib.sh\nindex f8e3733..c00452a 100644\n--- a/t/test-lib.sh\n+++ b/t/test-lib.sh\n@@ -51,6 +51,12 @@ then\n fi\n GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n \n+if ! test -r \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n+then\n+\techo 'error: you do not seem to have built git yet.' >&2\n+\texit 1\n+fi\n+\n . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n export PERL_PATH SHELL_PATH\n \n-- \n1.7.8.1.362.g5d6df.dirty\n"},{"id":"199286","messageId":"7v7grstlp9.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"1347901579-5545-2-git-send-email-artagnon@gmail.com","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-17T20:53:54Z","receivedAt":"2012-09-17T20:53:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> When tests were run without building git, the following error message\n> was displayed:\n>\n> .: 54: Can't open /path/to/git/source/t/../GIT-BUILD-OPTIONS\n>\n> Change this to display a more user-friendly error message:\n>\n> error: you do not seem to have built git yet.\n>\n> Signed-off-by: Ramkumar Ramachandra \n> ---\n> t/t0000-basic.sh | 10 ----------\n> t/test-lib.sh | 6 ++++++\n> 2 files changed, 6 insertions(+), 10 deletions(-)\n>\n> diff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\n> index ae6a3f0..08677df 100755\n> --- a/t/t0000-basic.sh\n> +++ b/t/t0000-basic.sh\n> @@ -18,16 +18,6 @@ swapping compression and hashing order, the person who is making the\n> modification *should* take notice and update the test vectors here.\n> '\n> \n> -################################################################\n> -# It appears that people try to run tests without building...\n> -\n> -../git >/dev/null\n> -if test $? != 1\n> -then\n> -\techo >&2 'You do not seem to have built git yet.'\n> -\texit 1\n> -fi\n> -\n> . ./test-lib.sh\n> \n> ################################################################\n> diff --git a/t/test-lib.sh b/t/test-lib.sh\n> index f8e3733..c00452a 100644\n> --- a/t/test-lib.sh\n> +++ b/t/test-lib.sh\n> @@ -51,6 +51,12 @@ then\n> fi\n> GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n> \n> +if ! test -r \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n> +then\n> +\techo 'error: you do not seem to have built git yet.' >&2\n> +\texit 1\n> +fi\n> +\n\nIs this a sufficient replacement for what you removed from 0000?\nCan the BUILD-OPTIONS file exist when your build of git failed?\n\n> . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n> export PERL_PATH SHELL_PATH\n"},{"id":"199295","messageId":"7vpq5ks5lx.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"1347901579-5545-1-git-send-email-artagnon@gmail.com","subject":"Re: [PATCH] t/perf: add \"trash directory\" to .gitignore","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-17T21:26:50Z","receivedAt":"2012-09-17T21:26:50Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> Signed-off-by: Ramkumar Ramachandra \n> ---\n> t/perf/.gitignore | 5 +++--\n> 1 files changed, 3 insertions(+), 2 deletions(-)\n>\n> diff --git a/t/perf/.gitignore b/t/perf/.gitignore\n> index 50f5cc1..0061cbc 100644\n> --- a/t/perf/.gitignore\n> +++ b/t/perf/.gitignore\n> @@ -1,2 +1,3 @@\n> -build/\n> -test-results/\n> +/build\n> +/test-results\n> +/trash directory*\n\nThanks. I _think_ you still want to make sure these are\ndirectories, so instead of losing the trailing slash, you would want\nto keep it and add a leading slash to anchor them to the t/perf\ndirectory, i.e.\n\n\t/build/\n /test-results/\n /trash directory*/\n"},{"id":"199336","messageId":"CALkWK0=zTZs6SLDq-FL4RYL+MZtJQAB1ApE0o7q5FqaieK01fQ@mail.gmail.com","threadId":"31570","inReplyTo":"7vpq5ks5lx.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] t/perf: add \"trash directory\" to .gitignore","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-18T06:36:32Z","receivedAt":"2012-09-18T06:36:32Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi,\n\nJunio C Hamano wrote:\n> Thanks. I _think_ you still want to make sure these are\n> directories, so instead of losing the trailing slash, you would want\n> to keep it and add a leading slash to anchor them to the t/perf\n> directory, i.e.\n>\n> /build/\n> /test-results/\n> /trash directory*/\n\nRight. Thanks for taking care of this.\n\nRam\n"},{"id":"199338","messageId":"CALkWK0k9P9-K5R2Amipujq2DFvJTXy8T3ZyfFa31CcxeKSFL2g@mail.gmail.com","threadId":"31570","inReplyTo":"7v7grstlp9.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-18T06:52:26Z","receivedAt":"2012-09-18T06:52:26Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi Junio,\n\nJunio C Hamano wrote:\n> Is this a sufficient replacement for what you removed from 0000?\n> Can the BUILD-OPTIONS file exist when your build of git failed?\n\nOops, I didn't realize that BUILD-OPTIONS would be written when the\nbuild fails. How about something like this instead:\n\ndiff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\nindex ae6a3f0..08677df 100755\n--- a/t/t0000-basic.sh\n+++ b/t/t0000-basic.sh\n@@ -18,16 +18,6 @@ swapping compression and hashing order, the person\nwho is making the\n modification *should* take notice and update the test vectors here.\n '\n\n-################################################################\n-# It appears that people try to run tests without building...\n-\n-../git >/dev/null\n-if test $? != 1\n-then\n-\techo >&2 'You do not seem to have built git yet.'\n-\texit 1\n-fi\n-\n . ./test-lib.sh\n\n ################################################################\n\tModified t/test-lib.sh\ndiff --git a/t/test-lib.sh b/t/test-lib.sh\nindex f8e3733..8cc7755 100644\n--- a/t/test-lib.sh\n+++ b/t/test-lib.sh\n@@ -51,6 +51,12 @@ then\n fi\n GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n\n+if ! test -x \"$GIT_BUILD_DIR\"/git || \"$GIT_BUILD_DIR\"/git && test $? != 1\n+then\n+\techo 'error: you do not seem to have built git yet.' >&2\n+\texit 1\n+fi\n+\n . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n export PERL_PATH SHELL_PATH\n"},{"id":"199341","messageId":"7v627bq0gx.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"CALkWK0k9P9-K5R2Amipujq2DFvJTXy8T3ZyfFa31CcxeKSFL2g@mail.gmail.com","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-18T07:00:46Z","receivedAt":"2012-09-18T07:00:46Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> Hi Junio,\n>\n> Junio C Hamano wrote:\n>> Is this a sufficient replacement for what you removed from 0000?\n>> Can the BUILD-OPTIONS file exist when your build of git failed?\n>\n> Oops, I didn't realize that BUILD-OPTIONS would be written when the\n> build fails. How about something like this instead:\n\nYeah, but why change it so much? Wouldn't writing\n\n\t\"$GIT_BUILD_DIR/git\" >/dev/null\n\tif test $? != 1\n then\n\t\t: You haven't built git!\n\tfi\n\njust like the original in 0000 be sufficient??\n\n> diff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\n> index ae6a3f0..08677df 100755\n> --- a/t/t0000-basic.sh\n> +++ b/t/t0000-basic.sh\n> @@ -18,16 +18,6 @@ swapping compression and hashing order, the person\n> who is making the\n> modification *should* take notice and update the test vectors here.\n> '\n>\n> -################################################################\n> -# It appears that people try to run tests without building...\n> -\n> -../git >/dev/null\n> -if test $? != 1\n> -then\n> -\techo >&2 'You do not seem to have built git yet.'\n> -\texit 1\n> -fi\n> -\n> . ./test-lib.sh\n>\n> ################################################################\n> \tModified t/test-lib.sh\n> diff --git a/t/test-lib.sh b/t/test-lib.sh\n> index f8e3733..8cc7755 100644\n> --- a/t/test-lib.sh\n> +++ b/t/test-lib.sh\n> @@ -51,6 +51,12 @@ then\n> fi\n> GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n>\n> +if ! test -x \"$GIT_BUILD_DIR\"/git || \"$GIT_BUILD_DIR\"/git && test $? != 1\n> +then\n> +\techo 'error: you do not seem to have built git yet.' >&2\n> +\texit 1\n> +fi\n> +\n> . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n> export PERL_PATH SHELL_PATH\n"},{"id":"199343","messageId":"CALkWK0kbeLtU-5_9QnmQKS1yO02SvCW3ERvami9bmCa0Vym8oQ@mail.gmail.com","threadId":"31570","inReplyTo":"7v627bq0gx.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-18T07:17:42Z","receivedAt":"2012-09-18T07:17:42Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi,\n\nJunio C Hamano wrote:\n> Ramkumar Ramachandra writes:\n>\n>> Hi Junio,\n>>\n>> Junio C Hamano wrote:\n>>> Is this a sufficient replacement for what you removed from 0000?\n>>> Can the BUILD-OPTIONS file exist when your build of git failed?\n>>\n>> Oops, I didn't realize that BUILD-OPTIONS would be written when the\n>> build fails. How about something like this instead:\n>\n> Yeah, but why change it so much? Wouldn't writing\n>\n> \"$GIT_BUILD_DIR/git\" >/dev/null\n> if test $? != 1\n> then\n> : You haven't built git!\n> fi\n>\n> just like the original in 0000 be sufficient??\n\nBecause that emits an ugly\n./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n\nRam\n"},{"id":"199348","messageId":"7vr4pzoj5a.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"CALkWK0kbeLtU-5_9QnmQKS1yO02SvCW3ERvami9bmCa0Vym8oQ@mail.gmail.com","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-18T08:00:17Z","receivedAt":"2012-09-18T08:00:17Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> Hi,\n>\n> Junio C Hamano wrote:\n>> Ramkumar Ramachandra writes:\n>>\n>>> Hi Junio,\n>>>\n>>> Junio C Hamano wrote:\n>>>> Is this a sufficient replacement for what you removed from 0000?\n>>>> Can the BUILD-OPTIONS file exist when your build of git failed?\n>>>\n>>> Oops, I didn't realize that BUILD-OPTIONS would be written when the\n>>> build fails. How about something like this instead:\n>>\n>> Yeah, but why change it so much? Wouldn't writing\n>>\n>> \"$GIT_BUILD_DIR/git\" >/dev/null\n>> if test $? != 1\n>> then\n>> : You haven't built git!\n>> fi\n>>\n>> just like the original in 0000 be sufficient??\n>\n> Because that emits an ugly\n> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n\nDon't you deserve it? ;-)\n\nThe full message would read\n\n ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n error: you do not seem to have built git yet.\n\nwhich looks perfectly sensible to me. It makes it clear where on\nthe filesystem the test script expects your \"git\", which is an added\nbenefit.\n"},{"id":"199349","messageId":"CALkWK0=JY3pTyyMAmFVnQatn6gd3eC7dxiBbTh0WmVwkdLTbHQ@mail.gmail.com","threadId":"31570","inReplyTo":"7vr4pzoj5a.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2012-09-18T08:07:57Z","receivedAt":"2012-09-18T08:07:57Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi Junio,\n\nJunio C Hamano wrote:\n>> Junio C Hamano wrote:\n>>> Yeah, but why change it so much? Wouldn't writing\n>>>\n>>> \"$GIT_BUILD_DIR/git\" >/dev/null\n>>> if test $? != 1\n>>> then\n>>> : You haven't built git!\n>>> fi\n>>>\n>>> just like the original in 0000 be sufficient??\n>>\n>> Because that emits an ugly\n>> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n>\n> Don't you deserve it? ;-)\n>\n> The full message would read\n>\n> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n> error: you do not seem to have built git yet.\n>\n> which looks perfectly sensible to me. It makes it clear where on\n> the filesystem the test script expects your \"git\", which is an added\n> benefit.\n\nFair enough- use your version then. Let me know if you want me to\nsubmit a revised patch.\n\nRam\n"},{"id":"199410","messageId":"7va9wnkq9u.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"1347901579-5545-2-git-send-email-artagnon@gmail.com","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-18T20:52:13Z","receivedAt":"2012-09-18T20:52:13Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> When tests were run without building git, the following error message\n> was displayed:\n>\n> .: 54: Can't open /path/to/git/source/t/../GIT-BUILD-OPTIONS\n\nDoes the test stop due to this error, or it just goes on and hit\nanother error? I am guessing that it is the latter, and if that is\nthe case, a more important change to describe here is\n\n> Change this to display a more user-friendly error message:\n>\n> error: you do not seem to have built git yet.\n\n... \"and stop the execution.\"\n\n> Signed-off-by: Ramkumar Ramachandra \n> ---\n> t/t0000-basic.sh | 10 ----------\n> t/test-lib.sh | 6 ++++++\n> 2 files changed, 6 insertions(+), 10 deletions(-)\n>\n> diff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\n> index ae6a3f0..08677df 100755\n> --- a/t/t0000-basic.sh\n> +++ b/t/t0000-basic.sh\n> @@ -18,16 +18,6 @@ swapping compression and hashing order, the person who is making the\n> modification *should* take notice and update the test vectors here.\n> '\n> \n> -################################################################\n> -# It appears that people try to run tests without building...\n> -\n> -../git >/dev/null\n> -if test $? != 1\n> -then\n> -\techo >&2 'You do not seem to have built git yet.'\n> -\texit 1\n> -fi\n> -\n> . ./test-lib.sh\n> \n> ################################################################\n> diff --git a/t/test-lib.sh b/t/test-lib.sh\n> index f8e3733..c00452a 100644\n> --- a/t/test-lib.sh\n> +++ b/t/test-lib.sh\n> @@ -51,6 +51,12 @@ then\n> fi\n> GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n> \n> +if ! test -r \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n> +then\n> +\techo 'error: you do not seem to have built git yet.' >&2\n> +\texit 1\n> +fi\n> +\n> . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n> export PERL_PATH SHELL_PATH\n"},{"id":"199411","messageId":"7v627bkq0h.fsf@alter.siamese.dyndns.org","threadId":"31570","inReplyTo":"CALkWK0=JY3pTyyMAmFVnQatn6gd3eC7dxiBbTh0WmVwkdLTbHQ@mail.gmail.com","subject":"Re: [PATCH] t/test-lib: print pretty msg when git isn't built","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-18T20:57:50Z","receivedAt":"2012-09-18T20:57:50Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n>>> Because that emits an ugly\n>>> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n>>\n>> Don't you deserve it? ;-)\n>>\n>> The full message would read\n>>\n>> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found\n>> error: you do not seem to have built git yet.\n>>\n>> which looks perfectly sensible to me. It makes it clear where on\n>> the filesystem the test script expects your \"git\", which is an added\n>> benefit.\n>\n> Fair enough- use your version then. Let me know if you want me to\n> submit a revised patch.\n\nOK, let's do this.\n\n-- >8 --\nFrom: Ramkumar Ramachandra \nDate: Mon, 17 Sep 2012 22:36:19 +0530\nSubject: [PATCH] t/test-lib: print pretty msg when git isn't built\n\nWhen tests were run without building git, they stopped with:\n\n .: 54: Can't open /path/to/git/source/t/../GIT-BUILD-OPTIONS\n\nMove the check that makes sure that git has already been built from\nt0000 to test-lib, so that any test will do so before it runs.\n\nSigned-off-by: Ramkumar Ramachandra \nSigned-off-by: Junio C Hamano \n---\n t/t0000-basic.sh | 10 ----------\n t/test-lib.sh | 7 +++++++\n 2 files changed, 7 insertions(+), 10 deletions(-)\n\ndiff --git a/t/t0000-basic.sh b/t/t0000-basic.sh\nindex ccb5435..741b6b6 100755\n--- a/t/t0000-basic.sh\n+++ b/t/t0000-basic.sh\n@@ -18,16 +18,6 @@ swapping compression and hashing order, the person who is making the\n modification *should* take notice and update the test vectors here.\n '\n \n-################################################################\n-# It appears that people try to run tests without building...\n-\n-../git >/dev/null\n-if test $? != 1\n-then\n-\techo >&2 'You do not seem to have built git yet.'\n-\texit 1\n-fi\n-\n . ./test-lib.sh\n \n ################################################################\ndiff --git a/t/test-lib.sh b/t/test-lib.sh\nindex 78c4286..61930fb 100644\n--- a/t/test-lib.sh\n+++ b/t/test-lib.sh\n@@ -51,6 +51,13 @@ then\n fi\n GIT_BUILD_DIR=\"$TEST_DIRECTORY\"/..\n \n+\"$GIT_BUILD_DIR/git\" >/dev/null\n+if test $? != 1\n+then\n+\techo >&2 'error: you do not seem to have built git yet.'\n+\texit 1\n+fi\n+\n . \"$GIT_BUILD_DIR\"/GIT-BUILD-OPTIONS\n export PERL_PATH SHELL_PATH\n \n-- \n1.7.12.563.g54818e4\n"}]}