Skip to content

Commit ec31f86

Browse files
committed
setlocalversion: absorb $(KERNELVERSION)
Print $(KERNELVERSION) in setlocalversion so that the callers get simpler. Signed-off-by: Masahiro Yamada <[email protected]>
1 parent 75280bd commit ec31f86

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

Makefile

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1258,8 +1258,7 @@ vmlinux: vmlinux.o $(KBUILD_LDS) modpost
12581258
# make sure no implicit rule kicks in
12591259
$(sort $(KBUILD_LDS) $(KBUILD_VMLINUX_OBJS) $(KBUILD_VMLINUX_LIBS)): . ;
12601260

1261-
filechk_kernel.release = \
1262-
echo "$(KERNELVERSION)$$($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion $(srctree))"
1261+
filechk_kernel.release = $(srctree)/scripts/setlocalversion $(srctree)
12631262

12641263
# Store (new) KERNELRELEASE string in include/config/kernel.release
12651264
include/config/kernel.release: FORCE
@@ -2124,7 +2123,7 @@ checkstack:
21242123
$(PERL) $(srctree)/scripts/checkstack.pl $(CHECKSTACK_ARCH)
21252124

21262125
kernelrelease:
2127-
@echo "$(KERNELVERSION)$$($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion $(srctree))"
2126+
@$(srctree)/scripts/setlocalversion $(srctree)
21282127

21292128
kernelversion:
21302129
@echo $(KERNELVERSION)

scripts/setlocalversion

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,11 @@ if ! test -e include/config/auto.conf; then
102102
exit 1
103103
fi
104104

105+
if [ -z "${KERNELVERSION}" ]; then
106+
echo "KERNELVERSION is not set" >&2
107+
exit 1
108+
fi
109+
105110
# localversion* files in the build and source directory
106111
res="$(collect_files localversion*)"
107112
if test ! "$srctree" -ef .; then
@@ -127,4 +132,4 @@ elif [ "${LOCALVERSION+set}" != "set" ]; then
127132
res="$res$(scm_version --short)"
128133
fi
129134

130-
echo "$res"
135+
echo "${KERNELVERSION}${res}"

0 commit comments

Comments
 (0)