The modules_install, firmware_install, all install commands
don't generate such hidden unused files under destination;
I think headers_install also should not to generate such .install
& ..install.cmd files;
Usually, we call "make headers_install INSTALL_HDR_PATH=/some/path"
expecting it install just the header files, not those dot files;
The original "call if_changed" could generate ..install.cmd and
together with "touch .install" could make the 2nd time headers_install
quite faster but it's not meaningful here, because we usually only install once;
diff --git a/scripts/Makefile.headersinst b/scripts/Makefile.headersinst
index a57f5bd..b3deff2 100644
--- a/scripts/Makefile.headersinst
+++ b/scripts/Makefile.headersinst
@@ -54,8 +54,7 @@ quiet_cmd_install = INSTALL $(printdir) ($(words
$(all-files))\
$(PERL) $< $(objtree)/$(obj) $(install) $(SRCARCH) $(objhdr-y); \
for F in $(wrapper-files); do \
echo "\#include <asm-generic/$$F>" > $(install)/$$F; \
- done; \
- touch $@
+ done
quiet_cmd_remove = REMOVE $(unwanted)
cmd_remove = rm -f $(unwanted-file)
@@ -66,8 +65,7 @@ quiet_cmd_check = CHECK $(printdir) ($(words
$(all-files)) files)
cmd_check = for f in $(all-files); do \
echo "$(install)/$${f}"; done \
| xargs \
- $(PERL) $< $(INSTALL_HDR_PATH)/include $(SRCARCH); \
- touch $@
+ $(PERL) $< $(INSTALL_HDR_PATH)/include $(SRCARCH)
PHONY += __headersinst __headerscheck
@@ -80,7 +78,7 @@ targets += $(install-file)
$(install-file): scripts/headers_install.pl $(input-files) FORCE
$(if $(unwanted),$(call cmd,remove),)
$(if $(wildcard $(dir $@)),,$(shell mkdir -p $(dir $@)))
- $(call if_changed,install)
+ $(call cmd,install)
else
__headerscheck: $(subdirs) $(check-file)
@@ -88,7 +86,7 @@ __headerscheck: $(subdirs) $(check-file)
targets += $(check-file)
$(check-file): scripts/headers_check.pl $(output-files) FORCE
- $(call if_changed,check)
+ $(call cmd,check)
endif
--
Cheng Renquan (程任全)
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
