2014-12-02 04:31:09

by Masahiro Yamada

[permalink] [raw]
Subject: [PATCH] kbuild: remove redundant -rR flag of hdr-inst

Passing -rR for "make headers_install" is redundant because
the top Makefile has already set -rR to MAKEFLAGS.

Signed-off-by: Masahiro Yamada <[email protected]>
---

scripts/Kbuild.include | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include
index eb14d83..c201ff3 100644
--- a/scripts/Kbuild.include
+++ b/scripts/Kbuild.include
@@ -193,10 +193,10 @@ modbuiltin := -f $(srctree)/scripts/Makefile.modbuiltin obj
clean := -f $(srctree)/scripts/Makefile.clean obj

###
-# Shorthand for $(Q)$(MAKE) -rR -f scripts/Makefile.headersinst obj=
+# Shorthand for $(Q)$(MAKE) -f scripts/Makefile.headersinst obj=
# Usage:
# $(Q)$(MAKE) $(hdr-inst)=dir
-hdr-inst := -rR -f $(srctree)/scripts/Makefile.headersinst obj
+hdr-inst := -f $(srctree)/scripts/Makefile.headersinst obj

# Prefix -I with $(srctree) if it is not an absolute path.
# skip if -I has no parameter
--
1.9.1


2014-12-03 13:22:14

by Michal Marek

[permalink] [raw]
Subject: Re: [PATCH] kbuild: remove redundant -rR flag of hdr-inst

On Tue, Dec 02, 2014 at 01:30:46PM +0900, Masahiro Yamada wrote:
> Passing -rR for "make headers_install" is redundant because
> the top Makefile has already set -rR to MAKEFLAGS.
>
> Signed-off-by: Masahiro Yamada <[email protected]>
> ---
>
> scripts/Kbuild.include | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

Thanks, applied to kbuild.git#kbuild.

Michal