{"thread":{"id":"31313","subject":"[PATCH] Document the --done option.","startedAt":"2012-08-22T10:57:05Z","lastAt":"2012-09-12T09:11:41Z","messageCount":5,"participants":["Eric S. Raymond","Junio C Hamano","Sverre Rabbelier"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"197575","messageId":"20120822105705.GA30472@thyrsus.com","threadId":"31313","inReplyTo":null,"subject":"[PATCH] Document the --done option.","fromName":"Eric S. Raymond","fromEmail":"esr@thyrsus.com","sentAt":"2012-08-22T10:57:05Z","receivedAt":"2012-08-22T10:57:05Z","isPatch":true,"sender":{"key":"esr@thyrsus.com","avatar":"https://avatars.githubusercontent.com/u/727961?v=4"},"body":"---\n Documentation/git-fast-import.txt |    8 +++++++-\n 1 file changed, 7 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt\nindex 2620d28..9291ea0 100644\n--- a/Documentation/git-fast-import.txt\n+++ b/Documentation/git-fast-import.txt\n@@ -39,6 +39,10 @@ OPTIONS\n \tSee ``Date Formats'' below for details about which formats\n \tare supported, and their syntax.\n \n+-- done::\n+\tTerminate with error if there is no 'done' command at the \n+\tend of the stream.\n+\n --force::\n \tForce updating modified existing branches, even if doing\n \tso would cause commits to be lost (as the new commit does\n@@ -1047,7 +1051,9 @@ done::\n \tError out if the stream ends without a 'done' command.\n \tWithout this feature, errors causing the frontend to end\n \tabruptly at a convenient point in the stream can go\n-\tundetected.\n+\tundetected.  This may occur, for example, if an import\n+\tfront end dies in mid-operation without emitting SIGTERM\n+\tor SIGKILL at its subordinate git fast-import instance.\n \n `option`\n ~~~~~~~~\n-- \n1.7.9.5\n\n\n-- \n\t\t<a href=\"http://www.catb.org/~esr/\">Eric S. Raymond</a>\n"},{"id":"197608","messageId":"7vobm2968x.fsf@alter.siamese.dyndns.org","threadId":"31313","inReplyTo":"20120822105705.GA30472@thyrsus.com","subject":"Re: [PATCH] Document the --done option.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-08-22T17:38:38Z","receivedAt":"2012-08-22T17:38:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Eric S. Raymond\" <esr@thyrsus.com> writes:\n\n> ---\n\nA forgotten Sign-off?\n\nSverre, the text matches my understanding as well as what be56862\n(fast-import: introduce 'done' command, 2011-07-16) says it did.\nAck?\n\n>  Documentation/git-fast-import.txt |    8 +++++++-\n>  1 file changed, 7 insertions(+), 1 deletion(-)\n>\n> diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt\n> index 2620d28..9291ea0 100644\n> --- a/Documentation/git-fast-import.txt\n> +++ b/Documentation/git-fast-import.txt\n> @@ -39,6 +39,10 @@ OPTIONS\n>  \tSee ``Date Formats'' below for details about which formats\n>  \tare supported, and their syntax.\n>  \n> +-- done::\n> +\tTerminate with error if there is no 'done' command at the \n> +\tend of the stream.\n> +\n>  --force::\n>  \tForce updating modified existing branches, even if doing\n>  \tso would cause commits to be lost (as the new commit does\n> @@ -1047,7 +1051,9 @@ done::\n>  \tError out if the stream ends without a 'done' command.\n>  \tWithout this feature, errors causing the frontend to end\n>  \tabruptly at a convenient point in the stream can go\n> -\tundetected.\n> +\tundetected.  This may occur, for example, if an import\n> +\tfront end dies in mid-operation without emitting SIGTERM\n> +\tor SIGKILL at its subordinate git fast-import instance.\n>  \n>  `option`\n>  ~~~~~~~~\n> -- \n> 1.7.9.5\n"},{"id":"197613","messageId":"CAGdFq_jTOY=+DJfuFdx8w8U-hQe9kFBLQDMaESpbeQKDafko2A@mail.gmail.com","threadId":"31313","inReplyTo":"7vobm2968x.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Document the --done option.","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2012-08-22T17:47:58Z","receivedAt":"2012-08-22T17:47:58Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"On Wed, Aug 22, 2012 at 10:38 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> \"Eric S. Raymond\" <esr@thyrsus.com> writes:\n>\n>> ---\n>\n> A forgotten Sign-off?\n>\n> Sverre, the text matches my understanding as well as what be56862\n> (fast-import: introduce 'done' command, 2011-07-16) says it did.\n> Ack?\n\nAcked-by: Sverre Rabbelier <srabbelier@gmail.com>\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"197800","messageId":"7voblzyj2q.fsf@alter.siamese.dyndns.org","threadId":"31313","inReplyTo":"7vobm2968x.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Document the --done option.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-08-24T23:19:09Z","receivedAt":"2012-08-24T23:19:09Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> \"Eric S. Raymond\" <esr@thyrsus.com> writes:\n>\n>> ---\n>\n> A forgotten Sign-off?\n\nPing?  Just telling us that this is Signed-off is fine.\n\nThanks.\n\n>\n> Sverre, the text matches my understanding as well as what be56862\n> (fast-import: introduce 'done' command, 2011-07-16) says it did.\n> Ack?\n>\n>>  Documentation/git-fast-import.txt |    8 +++++++-\n>>  1 file changed, 7 insertions(+), 1 deletion(-)\n>>\n>> diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt\n>> index 2620d28..9291ea0 100644\n>> --- a/Documentation/git-fast-import.txt\n>> +++ b/Documentation/git-fast-import.txt\n>> @@ -39,6 +39,10 @@ OPTIONS\n>>  \tSee ``Date Formats'' below for details about which formats\n>>  \tare supported, and their syntax.\n>>  \n>> +-- done::\n>> +\tTerminate with error if there is no 'done' command at the \n>> +\tend of the stream.\n>> +\n>>  --force::\n>>  \tForce updating modified existing branches, even if doing\n>>  \tso would cause commits to be lost (as the new commit does\n>> @@ -1047,7 +1051,9 @@ done::\n>>  \tError out if the stream ends without a 'done' command.\n>>  \tWithout this feature, errors causing the frontend to end\n>>  \tabruptly at a convenient point in the stream can go\n>> -\tundetected.\n>> +\tundetected.  This may occur, for example, if an import\n>> +\tfront end dies in mid-operation without emitting SIGTERM\n>> +\tor SIGKILL at its subordinate git fast-import instance.\n>>  \n>>  `option`\n>>  ~~~~~~~~\n>> -- \n>> 1.7.9.5\n"},{"id":"198818","messageId":"7vfw6nfvw2.fsf@alter.siamese.dyndns.org","threadId":"31313","inReplyTo":"7voblzyj2q.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Document the --done option.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-12T09:11:41Z","receivedAt":"2012-09-12T09:11:41Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> \"Eric S. Raymond\" <esr@thyrsus.com> writes:\n>>\n>>> ---\n>>\n>> A forgotten Sign-off?\n>\n> Ping?  Just telling us that this is Signed-off is fine.\n\nPing again.\n"}]}