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

Re: [PATCH] submodule.sh: remove unused variable

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Aug 16, 2017, 22:59 UTC
Message-ID
<20170816225920.GJ13924@aiede.mtv.corp.google.com>
In-Reply-To
<20170816225042.32765-1-sbeller@google.com>
Stefan Beller wrote:
Show 7 quoted lines
> This could have been part of 48308681b0 (git submodule update: have a
> dedicated helper for cloning, 2016-02-29).
>
> Signed-off-by: Stefan Beller <sbeller@google.com>
> ---
>  git-submodule.sh | 1 -
>  1 file changed, 1 deletion(-)
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Thanks for cleaning up.
Show 15 quoted lines
> diff --git a/git-submodule.sh b/git-submodule.sh
> index e131760eec..9dcec7b356 100755
> --- a/git-submodule.sh
> +++ b/git-submodule.sh
> @@ -611,7 +611,6 @@ cmd_update()
>  		die_if_unmatched "$mode" "$sha1"
>  
>  		name=$(git submodule--helper name "$sm_path") || exit
> -		url=$(git config submodule."$name".url)
>  		if ! test -z "$update"
>  		then
>  			update_module=$update
> -- 
> 2.14.0.rc0.3.g6c2e499285
> 
Previous: Stefan Beller
Message 2 of 2 in “submodule.sh: remove unused variable”
  1. submodule.sh: remove unused variableStefan Beller, Aug 16, 2017
  2. Jonathan NiederAug 16, 2017

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.