Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-11 Thread Mark Levedahl
On 08/08/2013 01:44 PM, Ramsay Jones wrote: Fredrik Gustafsson wrote: On Sun, Aug 04, 2013 at 07:34:48PM +0200, Jens Lehmann wrote: But we'll have to use sm_path here (like everywhere else in the submodule script), because we'll run into problems under Windows otherwise (see 64394e3ae9 for deta

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-09 Thread Junio C Hamano
Ramsay Jones writes: > $path is part of the public API, so we can't just remove it. It would > require a deprecation period, etc,. (Adding/documenting $sm_path as an > alternative *may* be worth doing. dunno.) I think exporting sm_path (if not done already) and documenting the transition may be

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-09 Thread Fredrik Gustafsson
On Thu, Aug 08, 2013 at 06:44:22PM +0100, Ramsay Jones wrote: > $path is part of the public API, so we can't just remove it. It would > require a deprecation period, etc,. (Adding/documenting $sm_path as an > alternative *may* be worth doing. dunno.) Maybe something for git 2.0? Well, Jens and Ju

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-09 Thread Ramsay Jones
Fredrik Gustafsson wrote: > On Sun, Aug 04, 2013 at 07:34:48PM +0200, Jens Lehmann wrote: >> But we'll have to use sm_path here (like everywhere else in the >> submodule script), because we'll run into problems under Windows >> otherwise (see 64394e3ae9 for details). Apart from that the patch >> is

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-06 Thread Jens Lehmann
Am 04.08.2013 23:29, schrieb Fredrik Gustafsson: > On Sun, Aug 04, 2013 at 07:34:48PM +0200, Jens Lehmann wrote: >> But we'll have to use sm_path here (like everywhere else in the >> submodule script), because we'll run into problems under Windows >> otherwise (see 64394e3ae9 for details). Apart fr

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-04 Thread Fredrik Gustafsson
On Sun, Aug 04, 2013 at 07:34:48PM +0200, Jens Lehmann wrote: > But we'll have to use sm_path here (like everywhere else in the > submodule script), because we'll run into problems under Windows > otherwise (see 64394e3ae9 for details). Apart from that the patch > is fine. We're still using path=

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-04 Thread Jens Lehmann
Am 03.08.2013 20:14, schrieb Jonathan Nieder: > brian m. carlson wrote: > >> cmd_summary reads the output of git diff, but reads in the submodule >> path into a variable called name. Since this variable does not >> contain the name of the submodule, but the path, rename it to be >> clearer what d

Re: [PATCH 1/2] submodule: fix confusing variable name

2013-08-03 Thread Jonathan Nieder
brian m. carlson wrote: > cmd_summary reads the output of git diff, but reads in the submodule > path into a variable called name. Since this variable does not > contain the name of the submodule, but the path, rename it to be > clearer what data it actually holds. Nice. Reviewed-by: Jonathan N

[PATCH 1/2] submodule: fix confusing variable name

2013-08-03 Thread brian m. carlson
cmd_summary reads the output of git diff, but reads in the submodule path into a variable called name. Since this variable does not contain the name of the submodule, but the path, rename it to be clearer what data it actually holds. Signed-off-by: brian m. carlson --- git-submodule.sh | 8