git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[RFC PATCH 1/2] submodule: move update configuration variable further up

From
Heiko Voigt <hvoigt@hvoigt.net>
Date
Jun 6, 2011, 20:57 UTC
Message-ID
<4764d04d0faf6d71830094f91bd3743deb6e2d4e.1307374519.git.hvoigt@hvoigt.net>
In-Reply-To
<cover.1307374519.git.hvoigt@hvoigt.net>
Lets always initialize the 'update_module' variable with the final
value. This way we allow code which wants to check this configuration
early to do so right in the beginning of cmd_update().
---
 git-submodule.sh |   13 +++++++------
 1 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/git-submodule.sh b/git-submodule.sh
index d189a24..70dfc7b 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh
@@ -454,7 +454,13 @@ cmd_update()
 		fi
 		name=$(module_name "$path") || exit
 		url=$(git config submodule."$name".url)
-		update_module=$(git config submodule."$name".update)
+		if ! test -z "$update"
+		then
+			update_module=$update
+		else
+			update_module=$(git config submodule."$name".update)
+		fi
+
 		if test -z "$url"
 		then
 			# Only mention uninitialized submodules when its
@@ -476,11 +482,6 @@ cmd_update()
 			die "Unable to find current revision in submodule path '$path'"
 		fi
 
-		if ! test -z "$update"
-		then
-			update_module=$update
-		fi
-
 		if test "$subsha1" != "$sha1"
 		then
 			subforce=$force
-- 
1.7.5.1.219.g4c6b2
Previous: Heiko VoigtNext: Heiko Voigt
Message 2 of 4 in “Add an update=none option for 'loose' submodules”
  1. 0/2 Add an update=none option for 'loose' submodulesHeiko Voigt, Jun 6, 2011
  2. 1/2 submodule: move update configuration variable further upHeiko Voigt, Jun 6, 2011
  3. 2/2 add update 'none' flag to disable update of submodule by defaultHeiko Voigt, Jun 6, 2011
  4. Jens LehmannJun 9, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.