From: Jani Nikula <jani.nikula@intel.com> To: Masahiro Yamada <yamada.masahiro@socionext.com>, linux-kbuild@vger.kernel.org Cc: Randy Dunlap <rdunlap@infradead.org>, Arnd Bergmann <arnd@arndb.de>, David Howells <dhowells@redhat.com>, Sam Ravnborg <sam@ravnborg.org>, Masahiro Yamada <yamada.masahiro@socionext.com>, Michal Marek <michal.lkml@markovi.net>, linux-kernel@vger.kernel.org Subject: Re: [PATCH 14/15] fixup: kbuild: add support for ensuring headers are self-contained Date: Tue, 04 Jun 2019 15:53:57 +0300 [thread overview] Message-ID: <871s094il6.fsf@intel.com> (raw) In-Reply-To: <20190604101409.2078-15-yamada.masahiro@socionext.com> 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
WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@intel.com> To: Masahiro Yamada <yamada.masahiro@socionext.com>, linux-kbuild@vger.kernel.org Cc: Randy Dunlap <rdunlap@infradead.org>, Arnd Bergmann <arnd@arndb.de>, David Howells <dhowells@redhat.com>, Sam Ravnborg <sam@ravnborg.org>, Michal Marek <michal.lkml@markovi.net>, linux-kernel@vger.kernel.org Subject: Re: [PATCH 14/15] fixup: kbuild: add support for ensuring headers are self-contained Date: Tue, 04 Jun 2019 15:53:57 +0300 [thread overview] Message-ID: <871s094il6.fsf@intel.com> (raw) In-Reply-To: <20190604101409.2078-15-yamada.masahiro@socionext.com> 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
next prev parent reply other threads:[~2019-06-04 12:50 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-04 10:13 [PATCH 00/15] kbuild: refactor headers_install and support compile-test of UAPI headers Masahiro Yamada 2019-06-04 10:13 ` Masahiro Yamada 2019-06-04 10:13 ` Masahiro Yamada 2019-06-04 10:13 ` Masahiro Yamada 2019-06-04 10:13 ` [PATCH 01/15] kbuild: remove headers_{install,check}_all Masahiro Yamada 2019-06-04 17:57 ` Sam Ravnborg 2019-06-04 10:13 ` [PATCH 02/15] kbuild: remove stale dependency between Documentation/ and headers_install Masahiro Yamada 2019-06-04 10:13 ` [PATCH 03/15] kbuild: make gdb_script depend on prepare0 instead of prepare Masahiro Yamada 2019-06-04 10:13 ` [PATCH 04/15] kbuild: fix Kconfig prompt of CONFIG_HEADERS_CHECK Masahiro Yamada 2019-06-04 10:13 ` [PATCH 05/15] kbuild: add CONFIG_HEADERS_INSTALL and loosen the dependency of samples Masahiro Yamada 2019-06-04 10:13 ` Masahiro Yamada 2019-06-04 10:13 ` Masahiro Yamada 2019-06-04 10:14 ` [PATCH 06/15] kbuild: remove build_unifdef target in scripts/Makefile Masahiro Yamada 2019-06-04 10:14 ` [PATCH 07/15] kbuild: build all prerequisite of headers_install simultaneously Masahiro Yamada 2019-06-04 10:14 ` [PATCH 08/15] kbuild: add 'headers' target to build up ready-to-install uapi headers Masahiro Yamada 2019-06-04 10:14 ` [PATCH 09/15] kbuild: re-implement Makefile.headersinst without directory descending Masahiro Yamada 2019-06-04 10:14 ` [PATCH 10/15] kbuild: move hdr-inst shorthand to top Makefile Masahiro Yamada 2019-06-04 10:14 ` [PATCH 11/15] kbuild: simplify scripts/headers_install.sh Masahiro Yamada 2019-06-04 10:14 ` [PATCH 12/15] kbuild: deb-pkg: do not run headers_check Masahiro Yamada 2019-06-04 10:14 ` [PATCH 13/15] kbuild: add support for ensuring headers are self-contained Masahiro Yamada 2019-06-04 12:42 ` [PATCH v2] " Jani Nikula 2019-06-04 17:25 ` Sam Ravnborg 2019-06-18 15:45 ` Masahiro Yamada 2019-06-04 10:14 ` [PATCH 14/15] fixup: " Masahiro Yamada 2019-06-04 12:53 ` Jani Nikula [this message] 2019-06-04 12:53 ` Jani Nikula 2019-06-04 10:14 ` [PATCH 15/15] kbuild: compile test UAPI headers to ensure they " Masahiro Yamada 2019-06-04 10:14 ` Masahiro Yamada 2019-06-21 19:05 ` Arnd Bergmann 2019-06-21 19:05 ` Arnd Bergmann 2019-06-22 5:12 ` Masahiro Yamada 2019-06-22 5:12 ` Masahiro Yamada 2019-06-22 5:39 ` Masahiro Yamada 2019-06-22 5:39 ` Masahiro Yamada 2019-06-05 2:37 ` [PATCH 00/15] kbuild: refactor headers_install and support compile-test of UAPI headers Masahiro Yamada 2019-06-05 2:37 ` Masahiro Yamada 2019-06-05 2:37 ` Masahiro Yamada 2019-06-05 2:37 ` Masahiro Yamada 2019-06-18 15:46 ` Masahiro Yamada 2019-06-18 15:46 ` Masahiro Yamada 2019-06-18 15:46 ` Masahiro Yamada 2019-06-18 15:46 ` Masahiro Yamada
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=871s094il6.fsf@intel.com \ --to=jani.nikula@intel.com \ --cc=arnd@arndb.de \ --cc=dhowells@redhat.com \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=michal.lkml@markovi.net \ --cc=rdunlap@infradead.org \ --cc=sam@ravnborg.org \ --cc=yamada.masahiro@socionext.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.