{"thread":{"id":"36058","subject":"[PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","startedAt":"2014-03-04T23:01:16Z","lastAt":"2014-03-11T17:34:43Z","messageCount":7,"participants":["Henri GEIST","Jens Lehmann","Heiko Voigt"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"236053","messageId":"1393974076.7891.27.camel@Naugrim","threadId":"36058","inReplyTo":null,"subject":"[PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Henri GEIST","fromEmail":"geist.henri@laposte.net","sentAt":"2014-03-04T23:01:16Z","receivedAt":"2014-03-04T23:01:16Z","isPatch":true,"sender":{"key":"geist.henri@laposte.net","avatar":"https://avatars.githubusercontent.com/u/42912271?v=4"},"body":"Permit to do a 'git clone --recursive' through git-gui.\n\nSigned-off-by: Henri GEIST <geist.henri@laposte.net>\n---\nI have set the default checkbox state to 'true' by default has all my gui users\nuse it all the time this way.\nBut as it change the default behavior you may prefer to set it to 'false' by\ndefault.\n\n git-gui/lib/choose_repository.tcl |   34 ++++++++++++++++++++++++++++++++--\n 1 file changed, 32 insertions(+), 2 deletions(-)\n\ndiff --git a/git-gui/lib/choose_repository.tcl b/git-gui/lib/choose_repository.tcl\nindex 3c10bc6..47d436b 100644\n--- a/git-gui/lib/choose_repository.tcl\n+++ b/git-gui/lib/choose_repository.tcl\n@@ -18,6 +18,7 @@ field local_path       {} ; # Where this repository is locally\n field origin_url       {} ; # Where we are cloning from\n field origin_name  origin ; # What we shall call 'origin'\n field clone_type hardlink ; # Type of clone to construct\n+field recursive      true ; # Recursive cloning flag\n field readtree_err        ; # Error output from read-tree (if any)\n field sorted_recent       ; # recent repositories (sorted)\n \n@@ -525,6 +526,11 @@ method _do_clone {} {\n \tforeach r $w_types {\n \t\tpack $r -anchor w\n \t}\n+\t${NS}::checkbutton $args.type_f.recursive \\\n+\t\t-text [mc \"Recursive (For submodules)\"] \\\n+\t\t-variable @recursive \\\n+\t\t-onvalue true -offvalue false\n+\tpack $args.type_f.recursive\n \tgrid $args.type_l $args.type_f -sticky new\n \n \tgrid columnconfigure $args 1 -weight 1\n@@ -952,6 +958,30 @@ method _do_clone_checkout {HEAD} {\n \tfileevent $fd readable [cb _readtree_wait $fd]\n }\n \n+method _do_validate_submodule_cloning {ok} {\n+\tif {$ok} {\n+\t\t$o_cons done $ok\n+\t\tset done 1\n+\t} else {\n+\t\t_clone_failed $this [mc \"Cannot clone submodules.\"]\n+\t}\n+}\n+\n+method _do_clone_submodules {} {\n+\tif {$recursive eq {true}} {\n+\t\tdestroy $w_body\n+\t\tset o_cons [console::embed \\\n+\t\t\t$w_body \\\n+\t\t\t[mc \"Cloning submodules\"]]\n+\t\tpack $w_body -fill both -expand 1 -padx 10\n+\t\t$o_cons exec \\\n+\t\t\t[list git submodule update --init --recursive] \\\n+\t\t\t[cb _do_validate_submodule_cloning]\n+\t} else {\n+\t\tset done 1\n+\t}\n+}\n+\n method _readtree_wait {fd} {\n \tset buf [read $fd]\n \t$o_cons update_meter $buf\n@@ -982,7 +1012,7 @@ method _readtree_wait {fd} {\n \t\tfconfigure $fd_ph -blocking 0 -translation binary -eofchar {}\n \t\tfileevent $fd_ph readable [cb _postcheckout_wait $fd_ph]\n \t} else {\n-\t\tset done 1\n+\t\t_do_clone_submodules $this\n \t}\n }\n \n@@ -996,7 +1026,7 @@ method _postcheckout_wait {fd_ph} {\n \t\t\thook_failed_popup post-checkout $pch_error 0\n \t\t}\n \t\tunset pch_error\n-\t\tset done 1\n+\t\t_do_clone_submodules $this\n \t\treturn\n \t}\n \tfconfigure $fd_ph -blocking 0\n-- \n1.7.9.3.369.gd715.dirty\n\n"},{"id":"236102","messageId":"5317662C.6010404@web.de","threadId":"36058","inReplyTo":"1393974076.7891.27.camel@Naugrim","subject":"Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-03-05T18:00:12Z","receivedAt":"2014-03-05T18:00:12Z","isPatch":true,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 05.03.2014 00:01, schrieb Henri GEIST:\n> Permit to do a 'git clone --recursive' through git-gui.\n\nI really like where this is heading!\n\nSome minor issues:\n\n- I think we should be more verbose in the commit message,\n  including that and why the default should be \"on\". Maybe\n  like this?\n\n  \"Permit to do a 'git clone --recursive' through git-gui.\n  Add a 'recursive' checkbox in the clone menu which allows\n  users to clone a repository and all its submodules in one\n  go (unless the 'update' flag is set to \"none\" in the\n  .gitmodules file for a submodule, in that case that\n  specific submodule is not cloned automatically).\n\n  Enable this new option per default, as most users want to\n  clone all submodules too when cloning the superproject\n  (This is currently not possible without leaving git gui\n  or adding a custom tool entry for that).\"\n\n\n- I'd rather change the button text from \"Recursive (For\n  submodules)\" to something like \"Recursively clone\n  submodules too\" or such.\n\n\n- Wouldn't it be easier to pass the '--recurse-submodules\"\n  option to the \"git clone\" call for the superproject instead\n  of adding the _do_clone_submodules() function doing a\n  subsequent \"git submodule update --init --recursive\"? That\n  is also be more future proof with respect to the autoclone\n  config option we have in mind (which would add that behavior\n  for \"git clone\" itself, making the call you added redundant).\n\n\n> Signed-off-by: Henri GEIST <geist.henri@laposte.net>\n> ---\n> I have set the default checkbox state to 'true' by default has all my gui users\n> use it all the time this way.\n> But as it change the default behavior you may prefer to set it to 'false' by\n> default.\n> \n>  git-gui/lib/choose_repository.tcl |   34 ++++++++++++++++++++++++++++++++--\n>  1 file changed, 32 insertions(+), 2 deletions(-)\n> \n> diff --git a/git-gui/lib/choose_repository.tcl b/git-gui/lib/choose_repository.tcl\n> index 3c10bc6..47d436b 100644\n> --- a/git-gui/lib/choose_repository.tcl\n> +++ b/git-gui/lib/choose_repository.tcl\n> @@ -18,6 +18,7 @@ field local_path       {} ; # Where this repository is locally\n>  field origin_url       {} ; # Where we are cloning from\n>  field origin_name  origin ; # What we shall call 'origin'\n>  field clone_type hardlink ; # Type of clone to construct\n> +field recursive      true ; # Recursive cloning flag\n>  field readtree_err        ; # Error output from read-tree (if any)\n>  field sorted_recent       ; # recent repositories (sorted)\n>  \n> @@ -525,6 +526,11 @@ method _do_clone {} {\n>  \tforeach r $w_types {\n>  \t\tpack $r -anchor w\n>  \t}\n> +\t${NS}::checkbutton $args.type_f.recursive \\\n> +\t\t-text [mc \"Recursive (For submodules)\"] \\\n> +\t\t-variable @recursive \\\n> +\t\t-onvalue true -offvalue false\n> +\tpack $args.type_f.recursive\n>  \tgrid $args.type_l $args.type_f -sticky new\n>  \n>  \tgrid columnconfigure $args 1 -weight 1\n> @@ -952,6 +958,30 @@ method _do_clone_checkout {HEAD} {\n>  \tfileevent $fd readable [cb _readtree_wait $fd]\n>  }\n>  \n> +method _do_validate_submodule_cloning {ok} {\n> +\tif {$ok} {\n> +\t\t$o_cons done $ok\n> +\t\tset done 1\n> +\t} else {\n> +\t\t_clone_failed $this [mc \"Cannot clone submodules.\"]\n> +\t}\n> +}\n> +\n> +method _do_clone_submodules {} {\n> +\tif {$recursive eq {true}} {\n> +\t\tdestroy $w_body\n> +\t\tset o_cons [console::embed \\\n> +\t\t\t$w_body \\\n> +\t\t\t[mc \"Cloning submodules\"]]\n> +\t\tpack $w_body -fill both -expand 1 -padx 10\n> +\t\t$o_cons exec \\\n> +\t\t\t[list git submodule update --init --recursive] \\\n> +\t\t\t[cb _do_validate_submodule_cloning]\n> +\t} else {\n> +\t\tset done 1\n> +\t}\n> +}\n> +\n>  method _readtree_wait {fd} {\n>  \tset buf [read $fd]\n>  \t$o_cons update_meter $buf\n> @@ -982,7 +1012,7 @@ method _readtree_wait {fd} {\n>  \t\tfconfigure $fd_ph -blocking 0 -translation binary -eofchar {}\n>  \t\tfileevent $fd_ph readable [cb _postcheckout_wait $fd_ph]\n>  \t} else {\n> -\t\tset done 1\n> +\t\t_do_clone_submodules $this\n>  \t}\n>  }\n>  \n> @@ -996,7 +1026,7 @@ method _postcheckout_wait {fd_ph} {\n>  \t\t\thook_failed_popup post-checkout $pch_error 0\n>  \t\t}\n>  \t\tunset pch_error\n> -\t\tset done 1\n> +\t\t_do_clone_submodules $this\n>  \t\treturn\n>  \t}\n>  \tfconfigure $fd_ph -blocking 0\n> \n"},{"id":"236143","messageId":"1394064956.7891.28.camel@Naugrim","threadId":"36058","inReplyTo":"5317662C.6010404@web.de","subject":"Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Henri GEIST","fromEmail":"geist.henri@laposte.net","sentAt":"2014-03-06T00:15:56Z","receivedAt":"2014-03-06T00:15:56Z","isPatch":true,"sender":{"key":"geist.henri@laposte.net","avatar":"https://avatars.githubusercontent.com/u/42912271?v=4"},"body":"Le mercredi 05 mars 2014 à 19:00 +0100, Jens Lehmann a écrit :\n> Am 05.03.2014 00:01, schrieb Henri GEIST:\n> > Permit to do a 'git clone --recursive' through git-gui.\n> \n> I really like where this is heading!\n> \n> Some minor issues:\n> \n> - I think we should be more verbose in the commit message,\n>   including that and why the default should be \"on\". Maybe\n>   like this?\n> \n>   \"Permit to do a 'git clone --recursive' through git-gui.\n>   Add a 'recursive' checkbox in the clone menu which allows\n>   users to clone a repository and all its submodules in one\n>   go (unless the 'update' flag is set to \"none\" in the\n>   .gitmodules file for a submodule, in that case that\n>   specific submodule is not cloned automatically).\n> \n>   Enable this new option per default, as most users want to\n>   clone all submodules too when cloning the superproject\n>   (This is currently not possible without leaving git gui\n>   or adding a custom tool entry for that).\"\n> \n\nOk for me.\n\n> \n> - I'd rather change the button text from \"Recursive (For\n>   submodules)\" to something like \"Recursively clone\n>   submodules too\" or such.\n> \n\nPerfect\n\n> \n> - Wouldn't it be easier to pass the '--recurse-submodules\"\n>   option to the \"git clone\" call for the superproject instead\n>   of adding the _do_clone_submodules() function doing a\n>   subsequent \"git submodule update --init --recursive\"? That\n>   is also be more future proof with respect to the autoclone\n>   config option we have in mind (which would add that behavior\n>   for \"git clone\" itself, making the call you added redundant).\n> \n\nThat is what I planned to do at beginning.\nBut git-gui never call git clone anywhere.\nIt make the clone step by step with a long and complicated list of\ncommands just like a Tcl rewrite of git-clone.\nHave a look on the function _do_clone2 in choose_repository.tcl.\n\nAs I suspect there should be a good reason for this that I did not\nunderstand I have choose to not refactoring it.\nAnd in fact looking in the code 'git clone --recursive' do nothing\nelse than calling 'git submodule update --init --recursive' like I\nhave done to complete this rewrite of 'git-clone'.\n\n\n> \n> > Signed-off-by: Henri GEIST <geist.henri@laposte.net>\n> > ---\n> > I have set the default checkbox state to 'true' by default has all my gui users\n> > use it all the time this way.\n> > But as it change the default behavior you may prefer to set it to 'false' by\n> > default.\n> > \n> >  git-gui/lib/choose_repository.tcl |   34 ++++++++++++++++++++++++++++++++--\n> >  1 file changed, 32 insertions(+), 2 deletions(-)\n> > \n> > diff --git a/git-gui/lib/choose_repository.tcl b/git-gui/lib/choose_repository.tcl\n> > index 3c10bc6..47d436b 100644\n> > --- a/git-gui/lib/choose_repository.tcl\n> > +++ b/git-gui/lib/choose_repository.tcl\n> > @@ -18,6 +18,7 @@ field local_path       {} ; # Where this repository is locally\n> >  field origin_url       {} ; # Where we are cloning from\n> >  field origin_name  origin ; # What we shall call 'origin'\n> >  field clone_type hardlink ; # Type of clone to construct\n> > +field recursive      true ; # Recursive cloning flag\n> >  field readtree_err        ; # Error output from read-tree (if any)\n> >  field sorted_recent       ; # recent repositories (sorted)\n> >  \n> > @@ -525,6 +526,11 @@ method _do_clone {} {\n> >  \tforeach r $w_types {\n> >  \t\tpack $r -anchor w\n> >  \t}\n> > +\t${NS}::checkbutton $args.type_f.recursive \\\n> > +\t\t-text [mc \"Recursive (For submodules)\"] \\\n> > +\t\t-variable @recursive \\\n> > +\t\t-onvalue true -offvalue false\n> > +\tpack $args.type_f.recursive\n> >  \tgrid $args.type_l $args.type_f -sticky new\n> >  \n> >  \tgrid columnconfigure $args 1 -weight 1\n> > @@ -952,6 +958,30 @@ method _do_clone_checkout {HEAD} {\n> >  \tfileevent $fd readable [cb _readtree_wait $fd]\n> >  }\n> >  \n> > +method _do_validate_submodule_cloning {ok} {\n> > +\tif {$ok} {\n> > +\t\t$o_cons done $ok\n> > +\t\tset done 1\n> > +\t} else {\n> > +\t\t_clone_failed $this [mc \"Cannot clone submodules.\"]\n> > +\t}\n> > +}\n> > +\n> > +method _do_clone_submodules {} {\n> > +\tif {$recursive eq {true}} {\n> > +\t\tdestroy $w_body\n> > +\t\tset o_cons [console::embed \\\n> > +\t\t\t$w_body \\\n> > +\t\t\t[mc \"Cloning submodules\"]]\n> > +\t\tpack $w_body -fill both -expand 1 -padx 10\n> > +\t\t$o_cons exec \\\n> > +\t\t\t[list git submodule update --init --recursive] \\\n> > +\t\t\t[cb _do_validate_submodule_cloning]\n> > +\t} else {\n> > +\t\tset done 1\n> > +\t}\n> > +}\n> > +\n> >  method _readtree_wait {fd} {\n> >  \tset buf [read $fd]\n> >  \t$o_cons update_meter $buf\n> > @@ -982,7 +1012,7 @@ method _readtree_wait {fd} {\n> >  \t\tfconfigure $fd_ph -blocking 0 -translation binary -eofchar {}\n> >  \t\tfileevent $fd_ph readable [cb _postcheckout_wait $fd_ph]\n> >  \t} else {\n> > -\t\tset done 1\n> > +\t\t_do_clone_submodules $this\n> >  \t}\n> >  }\n> >  \n> > @@ -996,7 +1026,7 @@ method _postcheckout_wait {fd_ph} {\n> >  \t\t\thook_failed_popup post-checkout $pch_error 0\n> >  \t\t}\n> >  \t\tunset pch_error\n> > -\t\tset done 1\n> > +\t\t_do_clone_submodules $this\n> >  \t\treturn\n> >  \t}\n> >  \tfconfigure $fd_ph -blocking 0\n> > \n> \n\n\n"},{"id":"236186","messageId":"5318CE14.1090000@web.de","threadId":"36058","inReplyTo":"1394064956.7891.28.camel@Naugrim","subject":"Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-03-06T19:35:48Z","receivedAt":"2014-03-06T19:35:48Z","isPatch":true,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 06.03.2014 01:15, schrieb Henri GEIST:\n> Le mercredi 05 mars 2014 à 19:00 +0100, Jens Lehmann a écrit :\n>> Am 05.03.2014 00:01, schrieb Henri GEIST:\n>> - Wouldn't it be easier to pass the '--recurse-submodules\"\n>>   option to the \"git clone\" call for the superproject instead\n>>   of adding the _do_clone_submodules() function doing a\n>>   subsequent \"git submodule update --init --recursive\"? That\n>>   is also be more future proof with respect to the autoclone\n>>   config option we have in mind (which would add that behavior\n>>   for \"git clone\" itself, making the call you added redundant).\n> \n> That is what I planned to do at beginning.\n> But git-gui never call git clone anywhere.\n> It make the clone step by step with a long and complicated list of\n> commands just like a Tcl rewrite of git-clone.\n> Have a look on the function _do_clone2 in choose_repository.tcl.\n\nYou're right, it does fetch followed by read-tree ... so my\nproposal doesn't make much sense here, sorry for bothering you\nwithout checking the source first.\n\n> As I suspect there should be a good reason for this that I did not\n> understand I have choose to not refactoring it.\n\nThat makes sense. Shawn, could you shed some light on why clone\nis coded again using plumbing in git gui instead of just calling\nthe clone command?\n\n> And in fact looking in the code 'git clone --recursive' do nothing\n> else than calling 'git submodule update --init --recursive' like I\n> have done to complete this rewrite of 'git-clone'.\n\nYep. Note to self: Port everything we add to clone to git gui too.\n"},{"id":"236210","messageId":"20140306220034.GA30645@sandbox-ub","threadId":"36058","inReplyTo":"5318CE14.1090000@web.de","subject":"Re: Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2014-03-06T22:00:34Z","receivedAt":"2014-03-06T22:00:34Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Thu, Mar 06, 2014 at 08:35:48PM +0100, Jens Lehmann wrote:\n> Am 06.03.2014 01:15, schrieb Henri GEIST:\n> > Le mercredi 05 mars 2014 à 19:00 +0100, Jens Lehmann a écrit :\n> >> Am 05.03.2014 00:01, schrieb Henri GEIST:\n> >> - Wouldn't it be easier to pass the '--recurse-submodules\"\n> >>   option to the \"git clone\" call for the superproject instead\n> >>   of adding the _do_clone_submodules() function doing a\n> >>   subsequent \"git submodule update --init --recursive\"? That\n> >>   is also be more future proof with respect to the autoclone\n> >>   config option we have in mind (which would add that behavior\n> >>   for \"git clone\" itself, making the call you added redundant).\n> > \n> > That is what I planned to do at beginning.\n> > But git-gui never call git clone anywhere.\n> > It make the clone step by step with a long and complicated list of\n> > commands just like a Tcl rewrite of git-clone.\n> > Have a look on the function _do_clone2 in choose_repository.tcl.\n> \n> You're right, it does fetch followed by read-tree ... so my\n> proposal doesn't make much sense here, sorry for bothering you\n> without checking the source first.\n> \n> > As I suspect there should be a good reason for this that I did not\n> > understand I have choose to not refactoring it.\n> \n> That makes sense. Shawn, could you shed some light on why clone\n> is coded again using plumbing in git gui instead of just calling\n> the clone command?\n\nI think because git gui is using plumbing everywhere, it is supposed to\nbe just another porcelain. And I guess that was an intended decision\nbecause porcelain might change its output and break git gui. At least\nthats what I inferred.\n\nCheers Heiko\n"},{"id":"236495","messageId":"1394536038.7891.59.camel@Naugrim","threadId":"36058","inReplyTo":"5317662C.6010404@web.de","subject":"Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Henri GEIST","fromEmail":"geist.henri@laposte.net","sentAt":"2014-03-11T11:07:18Z","receivedAt":"2014-03-11T11:07:18Z","isPatch":true,"sender":{"key":"geist.henri@laposte.net","avatar":"https://avatars.githubusercontent.com/u/42912271?v=4"},"body":"Le mercredi 05 mars 2014 à 19:00 +0100, Jens Lehmann a écrit :\n> Am 05.03.2014 00:01, schrieb Henri GEIST:\n> > Permit to do a 'git clone --recursive' through git-gui.\n> \n> I really like where this is heading!\n> \n> Some minor issues:\n> \n> - I think we should be more verbose in the commit message,\n>   including that and why the default should be \"on\". Maybe\n>   like this?\n> \n>   \"Permit to do a 'git clone --recursive' through git-gui.\n>   Add a 'recursive' checkbox in the clone menu which allows\n>   users to clone a repository and all its submodules in one\n>   go (unless the 'update' flag is set to \"none\" in the\n>   .gitmodules file for a submodule, in that case that\n>   specific submodule is not cloned automatically).\n> \n>   Enable this new option per default, as most users want to\n>   clone all submodules too when cloning the superproject\n>   (This is currently not possible without leaving git gui\n>   or adding a custom tool entry for that).\"\n> \n> \n> - I'd rather change the button text from \"Recursive (For\n>   submodules)\" to something like \"Recursively clone\n>   submodules too\" or such.\n> \n>\n\nPerfect.\nWould you like me to send the new version of the patch in this thread\nOr to make a new thread [patch v2] ?\n\n\n"},{"id":"236510","messageId":"531F4933.7080701@web.de","threadId":"36058","inReplyTo":"1394536038.7891.59.camel@Naugrim","subject":"Re: [PATCH/RFC] git-gui: Add a 'recursive' checkbox in the clone menu.","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-03-11T17:34:43Z","receivedAt":"2014-03-11T17:34:43Z","isPatch":true,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 11.03.2014 12:07, schrieb Henri GEIST:\n> Le mercredi 05 mars 2014 à 19:00 +0100, Jens Lehmann a écrit :\n>> Am 05.03.2014 00:01, schrieb Henri GEIST:\n>>> Permit to do a 'git clone --recursive' through git-gui.\n>>\n>> I really like where this is heading!\n>>\n>> Some minor issues:\n>>\n>> - I think we should be more verbose in the commit message,\n>>   including that and why the default should be \"on\". Maybe\n>>   like this?\n>>\n>>   \"Permit to do a 'git clone --recursive' through git-gui.\n>>   Add a 'recursive' checkbox in the clone menu which allows\n>>   users to clone a repository and all its submodules in one\n>>   go (unless the 'update' flag is set to \"none\" in the\n>>   .gitmodules file for a submodule, in that case that\n>>   specific submodule is not cloned automatically).\n>>\n>>   Enable this new option per default, as most users want to\n>>   clone all submodules too when cloning the superproject\n>>   (This is currently not possible without leaving git gui\n>>   or adding a custom tool entry for that).\"\n>>\n>>\n>> - I'd rather change the button text from \"Recursive (For\n>>   submodules)\" to something like \"Recursively clone\n>>   submodules too\" or such.\n>>\n>>\n> \n> Perfect.\n> Would you like me to send the new version of the patch in this thread\n> Or to make a new thread [patch v2] ?\n\nIt doesn't matter that much as long as you start the subject with\n\"[PATCH v2]\". But I believe you should send it to Pat and create\nthe diff from inside the git-gui directory so he can simply apply\nit to his repository.\n"}]}