From: Christian Marangi Date: Thu, 4 Dec 2025 18:11:17 +0000 (+0100) Subject: sdk: invert logic for GIT_BRANCH and GIT_TAGNAME X-Git-Url: http://git.openwrt.org/?a=commitdiff_plain;h=289172b00e7193df8a2afa1eee54b43148470698;p=openwrt%2Fopenwrt.git sdk: invert logic for GIT_BRANCH and GIT_TAGNAME There is currently a logic error with generating the base feed string. In the scenario where the commit of a Tag match the latest commit of a stable branch, the stable branch have priority over the tag name. This results in generating SDK that actually target the branch instead of the specific tag. To fix this invert the logic and give priority to the tag name. Fixes: #18583 Signed-off-by: Christian Marangi --- diff --git a/target/sdk/Makefile b/target/sdk/Makefile index 0e91778238..537e56382c 100644 --- a/target/sdk/Makefile +++ b/target/sdk/Makefile @@ -44,10 +44,10 @@ GIT_COMMIT:=$(shell git rev-parse HEAD 2>/dev/null) GIT_BRANCH:=$(filter-out master main HEAD,$(shell git rev-parse --abbrev-ref HEAD 2>/dev/null)) GIT_TAGNAME:=$(shell git show-ref --tags --dereference 2>/dev/null | sed -ne '/^$(GIT_COMMIT) / { s|^.*/||; s|\^.*||; p }') -BASE_FEED:=$(if $(GIT_URL),src-git --root=package base $(GIT_URL)$(if $(GIT_BRANCH),;$(GIT_BRANCH),$(if $(GIT_TAGNAME),;$(GIT_TAGNAME)))) +BASE_FEED:=$(if $(GIT_URL),src-git --root=package base $(GIT_URL)$(if $(GIT_TAGNAME),;$(GIT_TAGNAME),$(if $(GIT_BRANCH),;$(GIT_BRANCH)))) BASE_FEED:=$(if $(BASE_FEED),$(BASE_FEED),$(shell cd $(TOPDIR); LC_ALL=C git svn info 2>/dev/null | sed -ne 's/^URL: /src-gitsvn --root=package base /p')) BASE_FEED:=$(if $(BASE_FEED),$(BASE_FEED),$(shell cd $(TOPDIR); LC_ALL=C svn info 2>/dev/null | sed -ne 's/^URL: /src-svn --root=package base /p')) -BASE_FEED:=$(if $(BASE_FEED),$(BASE_FEED),src-git --root=package base $(PROJECT_GIT)/openwrt/openwrt.git$(if $(GIT_BRANCH),;$(GIT_BRANCH),$(if $(GIT_TAGNAME),;$(GIT_TAGNAME)))) +BASE_FEED:=$(if $(BASE_FEED),$(BASE_FEED),src-git --root=package base $(PROJECT_GIT)/openwrt/openwrt.git$(if $(GIT_TAGNAME),;$(GIT_TAGNAME),$(if $(GIT_BRANCH),;$(GIT_BRANCH)))) KDIR_BASE = $(patsubst $(TOPDIR)/%,%,$(LINUX_DIR)) KDIR_ARCHES = $(LINUX_KARCH)