[PATCH v4 3/3] submodule: add --dissociate option to add/update commands

2018-05-03 Thread Casey Fitzpatrick
Add --dissociate option to add and update commands, both clone helper commands that already have the --reference option --dissociate pairs with. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 10 +- builtin/submodule--helper.c

[PATCH v4 2/3] submodule: add --progress option to add command

2018-05-03 Thread Casey Fitzpatrick
, but it is not documented. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 7 +++ git-submodule.sh| 5 - t/t7400-submodule-basic.sh | 16 3 files changed, 27 insertions(+), 1 deletion(-) diff --git a/Documentati

[PATCH v4 0/3] Add --progress and --dissociate to git submodule

2018-05-03 Thread Casey Fitzpatrick
for independent clones rather than depending on the reference. This is a resubmission with comments from Stefan Beller, Eric Sunshine, and Junio C Hamano addressed. Casey Fitzpatrick (3): submodule: clean up subsititions in script submodule: add --progress option to add command submodule: add

[PATCH v4 1/3] submodule: clean up subsititions in script

2018-05-03 Thread Casey Fitzpatrick
by using a substitution. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- git-submodule.sh | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/git-submodule.sh b/git-submodule.sh index 24914963c..262547968 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@

Re: [PATCH 1/3] submodule: clean up subsititions in script

2018-05-03 Thread Casey Fitzpatrick
Thanks I will amend and re-submit (this time with -v$N, I apologize for creating a confusing mess in everyone's email clients :)) On Wed, May 2, 2018 at 1:59 AM, Junio C Hamano <gits...@pobox.com> wrote: > Casey Fitzpatrick <kcgh...@gmail.com> writes: > >> 'reco

Re: [PATCH 0/3] Add --progress and --dissociate to git submodule

2018-05-02 Thread Casey Fitzpatrick
Thanks I was not aware of that option. On Wed, May 2, 2018 at 12:37 AM, Junio C Hamano <gits...@pobox.com> wrote: > Casey Fitzpatrick <kcgh...@gmail.com> writes: > >> These patches add --progress and --dissociate options to git submodule. >> >> The --progress

[PATCH 1/3] submodule: clean up subsititions in script

2018-05-01 Thread Casey Fitzpatrick
-by: Casey Fitzpatrick <kcgh...@gmail.com> --- git-submodule.sh | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/git-submodule.sh b/git-submodule.sh index 24914963c..262547968 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -465,7 +465,7 @@ cmd_

[PATCH 3/3] submodule: add --dissociate option to add/update commands

2018-05-01 Thread Casey Fitzpatrick
Add --dissociate option to add and update commands, both clone helper commands that already have the --reference option --dissociate pairs with. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 10 +- builtin/submodule--helper.c

[PATCH 2/3] submodule: add --progress option to add command

2018-05-01 Thread Casey Fitzpatrick
, but it is not documented. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 7 +++ git-submodule.sh| 5 - t/t7400-submodule-basic.sh | 16 3 files changed, 27 insertions(+), 1 deletion(-) diff --git a/Documentati

[PATCH 0/3] Add --progress and --dissociate to git submodule

2018-05-01 Thread Casey Fitzpatrick
for independent clones rather than depending on the reference. This is a resubmission with comments from Stefan Beller and Eric Sunshine addressed. Casey Fitzpatrick (3): submodule: clean up subsititions in script submodule: add --progress option to add command submodule: add --dissociate option

Re: [PATCH 3/3] submodule: add --dissociate option to add/update commands

2018-05-01 Thread Casey Fitzpatrick
Just noticed I missed the other 'test_must_fail'. Resubmitting in a few moments. On Tue, May 1, 2018 at 8:27 PM, Casey Fitzpatrick <kcgh...@gmail.com> wrote: > Add --dissociate option to add and update commands, both clone helper commands > that already have the --reference option

[PATCH 2/3] submodule: add --progress option to add command

2018-05-01 Thread Casey Fitzpatrick
, but it is not documented. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 7 +++ git-submodule.sh| 5 - t/t7400-submodule-basic.sh | 16 3 files changed, 27 insertions(+), 1 deletion(-) diff --git a/Documentati

[PATCH 3/3] submodule: add --dissociate option to add/update commands

2018-05-01 Thread Casey Fitzpatrick
Add --dissociate option to add and update commands, both clone helper commands that already have the --reference option --dissociate pairs with. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 10 +- builtin/submodule--helper.c

[PATCH 1/3] submodule: clean up subsititions in script

2018-05-01 Thread Casey Fitzpatrick
'recommend_shallow' and 'jobs' variables do not need quotes (they never contain spaces) and do not require any additional prefix, therefore remove the unnecessary subsitition. 'progress' is a boolean value. Treat it like the other boolean values in the script by using a substitution. ---

[PATCH 0/3] Add --progress and --dissociate to git submodule

2018-05-01 Thread Casey Fitzpatrick
for independent clones rather than depending on the reference. This is a resubmission with comments from Stefan Beller and Eric Sunshine addressed. Casey Fitzpatrick (3): submodule: clean up subsititions in script submodule: add --progress option to add command submodule: add --dissociate option

Re: [PATCH 2/2] submodule: Add --dissociate option to add/update commands

2018-05-01 Thread Casey Fitzpatrick
c Sunshine <sunsh...@sunshineco.com> wrote: > On Tue, May 1, 2018 at 2:09 PM, Casey Fitzpatrick <kcgh...@gmail.com> wrote: >> submodule: Add --dissociate option to add/update commands > > s/Add/add/ > >> Add --dissociate option to add and update commands, both

Re: [PATCH 1/2] submodule: Add --progress option to add command

2018-05-01 Thread Casey Fitzpatrick
y 1, 2018 at 11:09 AM, Casey Fitzpatrick <kcgh...@gmail.com> wrote: >> --progress was missing from add command, was only in update. >> Add --progress where it makes sense (both clone helper commands). >> Add missing documentation entry. >> Add test. > > Maybe: >

[PATCH 2/2] submodule: Add --dissociate option to add/update commands

2018-05-01 Thread Casey Fitzpatrick
Add --dissociate option to add and update commands, both clone helper commands that already have the --reference option --dissociate pairs with. Add documentation. Add tests. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 10 +- b

[PATCH 1/2] submodule: Add --progress option to add command

2018-05-01 Thread Casey Fitzpatrick
--progress was missing from add command, was only in update. Add --progress where it makes sense (both clone helper commands). Add missing documentation entry. Add test. Signed-off-by: Casey Fitzpatrick <kcgh...@gmail.com> --- Documentation/git-submodule.txt | 7 +++ git-submod

[PATCH 0/2] Add --progress and --dissociate to git submodule

2018-05-01 Thread Casey Fitzpatrick
for independent clones rather than depending on the reference. I apologize for any errors I may have made in creation of these patches or the formatting of these emails. This is the first time I have submitted patches to a mailing list. Casey Fitzpatrick (2): submodule: Add --progress option to add

Re: git-submodule is missing --dissociate option

2018-04-30 Thread Casey Fitzpatrick
Sure, I'll take a crack at it and submit a patch. On Mon, Apr 30, 2018 at 2:19 PM, Stefan Beller <sbel...@google.com> wrote: > On Mon, Apr 30, 2018 at 1:29 AM, Casey Fitzpatrick <kcgh...@gmail.com> wrote: >> This seems to be a hole in the git feature set. I believe it is fa

Re: git-submodule is missing --dissociate option

2018-04-30 Thread Casey Fitzpatrick
It also seems to be missing "--progress", and I imagine others. Perhaps submodule add/update should be reworked to automatically accept all the options that clone would? On Mon, Apr 30, 2018 at 4:29 AM, Casey Fitzpatrick <kcgh...@gmail.com> wrote: > This seems to be a hole in

git-submodule is missing --dissociate option

2018-04-30 Thread Casey Fitzpatrick
This seems to be a hole in the git feature set. I believe it is fairly easily worked around, but it would be best to provide the option for ease of use (and maybe performance?). git clone has both a --reference feature and a --dissociate option, with dissociate allowing for a reference to *only*