Skip to content

Commit

Permalink
submodule: fix confusing variable name
Browse files Browse the repository at this point in the history
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 <sandals@crustytoothpaste.net>
Acked-by: Jens Lehmann <Jens.Lehmann@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Brian M. Carlson authored and Junio C Hamano committed Sep 3, 2013
1 parent 4d06473 commit 2be9450
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions git-submodule.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1032,13 +1032,13 @@ cmd_summary() {
# Get modified modules cared by user
modules=$(git $diff_cmd $cached --ignore-submodules=dirty --raw $head -- "$@" |
sane_egrep '^:([0-7]* )?160000' |
while read mod_src mod_dst sha1_src sha1_dst status name
while read mod_src mod_dst sha1_src sha1_dst status sm_path
do
# Always show modules deleted or type-changed (blob<->module)
test $status = D -o $status = T && echo "$name" && continue
test $status = D -o $status = T && echo "$sm_path" && continue
# Also show added or modified modules which are checked out
GIT_DIR="$name/.git" git-rev-parse --git-dir >/dev/null 2>&1 &&
echo "$name"
GIT_DIR="$sm_path/.git" git-rev-parse --git-dir >/dev/null 2>&1 &&
echo "$sm_path"
done
)

Expand Down

0 comments on commit 2be9450

Please sign in to comment.