[PATCH] kbuild: Do not overwrite include/config/kernel.release needlessly

[PATCH] kbuild: Do not overwrite include/config/kernel.release needlessly

[Michal Marek]

Use filechk to detect if the content changed or not.

Signed-off-by: Michal Marek <mmarek@suse.cz>
---
 Makefile | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 4e3575c..3aa232f 100644
--- a/Makefile
+++ b/Makefile
@@ -794,10 +794,13 @@ PHONY += $(vmlinux-dirs)
 $(vmlinux-dirs): prepare scripts
 	$(Q)$(MAKE) $(build)=$@
 
+define filechk_kernel.release
+	echo "$(KERNELVERSION)$$($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion $(srctree))"
+endef
+
 # Store (new) KERNELRELEASE string in include/config/kernel.release
 include/config/kernel.release: include/config/auto.conf FORCE
-	$(Q)rm -f $@
-	$(Q)echo "$(KERNELVERSION)$$($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion $(srctree))" > $@
+	$(call filechk,kernel.release)
 
 
 # Things we need to do before we recursively start building the kernel
-- 
1.8.1.4

Re: [PATCH] kbuild: Do not overwrite include/config/kernel.release needlessly

[Michal Marek]

On 27.8.2013 15:53, Michal Marek wrote:
> Use filechk to detect if the content changed or not.
> 
> Signed-off-by: Michal Marek <mmarek@suse.cz>
> ---
>  Makefile | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)

I merged this to kbuild.git#kbuild now.

Thanks,
Michal