[PATCH] setlocalversion: make tag local to try_tag()

Yuntao Wang posted 1 patch 1 month ago
scripts/setlocalversion | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
[PATCH] setlocalversion: make tag local to try_tag()
Posted by Yuntao Wang 1 month ago
The tag variable is only used by try_tag(), but is currently declared
local to scm_version() and modified by try_tag().

Make tag local to try_tag() instead and remove the unused declaration
from scm_version().

No functional changes.

Signed-off-by: Yuntao Wang <yuntao.wang@linux.dev>
---
 scripts/setlocalversion | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/scripts/setlocalversion b/scripts/setlocalversion
index 28169d7e143b..f264c2460ad7 100755
--- a/scripts/setlocalversion
+++ b/scripts/setlocalversion
@@ -33,7 +33,7 @@ if test $# -gt 0 -o ! -d "$srctree"; then
 fi
 
 try_tag() {
-	tag="$1"
+	local tag="$1"
 
 	# Is $tag an annotated tag?
 	if [ "$(git cat-file -t "$tag" 2> /dev/null)" != tag ]; then
@@ -59,7 +59,6 @@ scm_version()
 {
 	local short=false
 	local no_dirty=false
-	local tag
 
 	while [ $# -gt 0 ];
 	do
-- 
2.55.0
Re: [PATCH] setlocalversion: make tag local to try_tag()
Posted by Nathan Chancellor 3 weeks, 6 days ago
On Sun, 23 Aug 2026 17:49:13 +0800, Yuntao Wang wrote:
> setlocalversion: make tag local to try_tag()

Applied to

  https://git.kernel.org/pub/scm/linux/kernel/git/kbuild/linux.git kbuild-next-unstable

Thanks!

[1/1] setlocalversion: make tag local to try_tag()
      https://git.kernel.org/kbuild/c/ba0e6a6f0fe7a

Please look out for regression or issue reports or other follow up
comments, as they may result in the patch/series getting dropped or
reverted. Patches applied to an "unstable" branch are accepted pending
wider testing in -next and any post-commit review; they will generally
be moved to the main branch in a week if no issues are found.

Best regards,
-- 
Cheers,
Nathan
Re: [PATCH] setlocalversion: make tag local to try_tag()
Posted by Nicolas Schier 1 month ago
On Sun, Aug 23, 2026 at 05:49:13PM +0800, Yuntao Wang wrote:
> The tag variable is only used by try_tag(), but is currently declared
> local to scm_version() and modified by try_tag().
> 
> Make tag local to try_tag() instead and remove the unused declaration
> from scm_version().
> 
> No functional changes.
> 
> Signed-off-by: Yuntao Wang <yuntao.wang@linux.dev>
> ---
>  scripts/setlocalversion | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 

Reviewed-by: Nicolas Schier <nsc@kernel.org>

-- 
Nicolas