{"thread":{"id":"36542","subject":"[PATCH v2] Document RUN_SETUP_GENTLY","startedAt":"2014-04-29T19:21:34Z","lastAt":"2014-04-30T18:29:59Z","messageCount":2,"participants":["David Turner","Junio C Hamano"],"isPatch":true,"patchVersion":2,"patchTotal":null},"messages":[{"id":"240199","messageId":"CAE+yK_kwMWRBDEVO=WVh5H-uFQJa2LEezRwbKnduLqoFjWC4zA@mail.gmail.com","threadId":"36542","inReplyTo":null,"subject":"[PATCH v2] Document RUN_SETUP_GENTLY","fromName":"David Turner","fromEmail":"dturner@twopensource.com","sentAt":"2014-04-29T19:21:34Z","receivedAt":"2014-04-29T19:21:34Z","isPatch":true,"sender":{"key":"novalis@novalis.org","avatar":"https://avatars.githubusercontent.com/u/77003?v=4"},"body":"Sorry about that -- the documentation of RUN_SETUP confused me.  So I\nhave a new patch that edits that as well.\n\n--\nRUN_SETUP_GENTLY and improve RUN_SETUP docs\n\nSigned-off-by: David Turner <dturner@twitter.com>\n---\n Documentation/technical/api-builtin.txt | 13 +++++++++----\n 1 file changed, 9 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/technical/api-builtin.txt\nb/Documentation/technical/api-builtin.txt\nindex e3d6e7a..b250c1a 100644\n--- a/Documentation/technical/api-builtin.txt\n+++ b/Documentation/technical/api-builtin.txt\n@@ -23,10 +23,15 @@ where options is the bitwise-or of:\n\n `RUN_SETUP`::\n\n- Make sure there is a Git directory to work on, and if there is a\n- work tree, chdir to the top of it if the command was invoked\n- in a subdirectory.  If there is no work tree, no chdir() is\n- done.\n+ If there is not a Git directory to work on, abort.  If there\n+ is a work tree, chdir to the top of it if the command was\n+ invoked in a subdirectory.  If there is no work tree, no\n+ chdir() is done.\n+\n+`RUN_SETUP_GENTLY`::\n+\n+ If there is a Git directory, chdir as per RUN_SETUP, otherwise,\n+ don't chdir anywhere.\n\n `USE_PAGER`::\n\n-- \n\n\nOn Tue, Apr 29, 2014 at 2:25 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> David Turner <dturner@twopensource.com> writes:\n>\n>> Document RUN_SETUP_GENTLY\n>>\n>> Signed-off-by: David Turner <dturner@twitter.com>\n>> ---\n>>  Documentation/technical/api-builtin.txt | 5 +++++\n>>  1 file changed, 5 insertions(+)\n>>\n>> diff --git a/Documentation/technical/api-builtin.txt\n>> b/Documentation/technical/api-builtin.txt\n>> index e3d6e7a..1bbeda2 100644\n>> --- a/Documentation/technical/api-builtin.txt\n>> +++ b/Documentation/technical/api-builtin.txt\n>> @@ -28,6 +28,11 @@ where options is the bitwise-or of:\n>>   in a subdirectory.  If there is no work tree, no chdir() is\n>>   done.\n>>\n>> +`RUN_SETUP_GENTLY`::\n>> +\n>> + If there is a Git directory, chdir as per RUN_SETUP, otherwise,\n>> + don't create one and don't chdir.\n>\n> I can understand \"don't chdir\" part, but where does \"don't create\"\n> come from?  Makes it sound as if non GENTLY version would create one\n> if there isn't there, but I am guessing that that is not what you\n> meant to say.\n>\n> Puzzled.\n>\n"},{"id":"240345","messageId":"xmqqeh0efz6g.fsf@gitster.dls.corp.google.com","threadId":"36542","inReplyTo":"CAE+yK_kwMWRBDEVO=WVh5H-uFQJa2LEezRwbKnduLqoFjWC4zA@mail.gmail.com","subject":"Re: [PATCH v2] Document RUN_SETUP_GENTLY","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-30T18:29:59Z","receivedAt":"2014-04-30T18:29:59Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"David Turner <dturner@twopensource.com> writes:\n\n> Sorry about that -- the documentation of RUN_SETUP confused me.  So I\n> have a new patch that edits that as well.\n>\n> --\n> RUN_SETUP_GENTLY and improve RUN_SETUP docs\n>\n> Signed-off-by: David Turner <dturner@twitter.com>\n> ---\n\nYou do not want to have \"Sorry about that...\" in your final commit\nlog message.  Please do the above this way if you wanted to:\n\n\n\tSorry about that ...\n\n\t-- >8 --\n\tSubject: docs: document RUN_SETUP_GENTLY and clarify RUN_SETUP\n\n\tWe only said what happens when we find the Git directory under\n\tRUN_SETUP, without saying what happens otherwise.\n\n\tSigned-off-by: ...\n\t---\n        $DIFFSTAT\n        $PATCH\n\n\n\n>  Documentation/technical/api-builtin.txt | 13 +++++++++----\n>  1 file changed, 9 insertions(+), 4 deletions(-)\n>\n> diff --git a/Documentation/technical/api-builtin.txt\n> b/Documentation/technical/api-builtin.txt\n> index e3d6e7a..b250c1a 100644\n> --- a/Documentation/technical/api-builtin.txt\n> +++ b/Documentation/technical/api-builtin.txt\n> @@ -23,10 +23,15 @@ where options is the bitwise-or of:\n>\n>  `RUN_SETUP`::\n>\n> - Make sure there is a Git directory to work on, and if there is a\n> - work tree, chdir to the top of it if the command was invoked\n\nYour mailer ate all the whitespaces and folded the lines, it seems.\n\nI'll fix it up and queue the result on 'pu', so please eyeball it\nand holler if I made a silly mistake while doing the fixup.\n\nThanks.\n\n\n> - in a subdirectory.  If there is no work tree, no chdir() is\n> - done.\n> + If there is not a Git directory to work on, abort.  If there\n> + is a work tree, chdir to the top of it if the command was\n> + invoked in a subdirectory.  If there is no work tree, no\n> + chdir() is done.\n> +\n> +`RUN_SETUP_GENTLY`::\n> +\n> + If there is a Git directory, chdir as per RUN_SETUP, otherwise,\n> + don't chdir anywhere.\n>\n>  `USE_PAGER`::\n"}]}