diff mbox series

[14/15] fixup: kbuild: add support for ensuring headers are self-contained

Message ID 20190604101409.2078-15-yamada.masahiro@socionext.com
State New
Headers show
Series kbuild: refactor headers_install and support compile-test of UAPI headers | expand

Commit Message

Masahiro Yamada June 4, 2019, 10:14 a.m. UTC
This is needed to avoid regeneration of header-test C files.

This will go away when Jani send v2.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

---

 scripts/Makefile.build | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

-- 
2.17.1

Comments

Jani Nikula June 4, 2019, 12:53 p.m. UTC | #1
On Tue, 04 Jun 2019, Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
> This is needed to avoid regeneration of header-test C files.

>

> This will go away when Jani send v2.


Done, in reply to patch 13.

Thanks,
Jani.


>

> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

> ---

>

>  scripts/Makefile.build | 5 ++---

>  1 file changed, 2 insertions(+), 3 deletions(-)

>

> diff --git a/scripts/Makefile.build b/scripts/Makefile.build

> index 2b4d56483c2e..067e98628664 100644

> --- a/scripts/Makefile.build

> +++ b/scripts/Makefile.build

> @@ -298,10 +298,9 @@ $(obj)/%.lst: $(src)/%.c FORCE

>  # ---------------------------------------------------------------------------

>  

>  quiet_cmd_header_test = HDRTEST $@

> -      cmd_header_test = echo "\#include \"$(<F)\"" > $@

> +      cmd_header_test = echo "\#include \"$*.h\"" > $@

>  

> -# FIXME: would be nice to be able to limit this implicit rule to header-test-y

> -$(obj)/%.header_test.c: $(src)/%.h FORCE

> +$(obj)/%.header_test.c:

>  	$(call if_changed,header_test)

>  

>  # Compile assembler sources (.S)


-- 
Jani Nikula, Intel Open Source Graphics Center
diff mbox series

Patch

diff --git a/scripts/Makefile.build b/scripts/Makefile.build
index 2b4d56483c2e..067e98628664 100644
--- a/scripts/Makefile.build
+++ b/scripts/Makefile.build
@@ -298,10 +298,9 @@  $(obj)/%.lst: $(src)/%.c FORCE
 # ---------------------------------------------------------------------------
 
 quiet_cmd_header_test = HDRTEST $@
-      cmd_header_test = echo "\#include \"$(<F)\"" > $@
+      cmd_header_test = echo "\#include \"$*.h\"" > $@
 
-# FIXME: would be nice to be able to limit this implicit rule to header-test-y
-$(obj)/%.header_test.c: $(src)/%.h FORCE
+$(obj)/%.header_test.c:
 	$(call if_changed,header_test)
 
 # Compile assembler sources (.S)