Re: [PATCH] kbuild: drop $(objtree)/ prefix support for clean-files
On Sun, May 1, 2022 at 8:57 PM Michael Ellerman wrote: > > Masahiro Yamada writes: > > I think this hack is a bad idea. arch/powerpc/boot/Makefile is the > > only user. Let's stop doing this. > > > > Signed-off-by: Masahiro Yamada > > --- > > > > arch/powerpc/boot/Makefile | 4 ++-- > > Acked-by: Michael Ellerman (powerpc) > > cheers > Applied to linux-kbuild. > > scripts/Makefile.clean | 8 +--- > > 2 files changed, 3 insertions(+), 9 deletions(-) > > > > diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile > > index 4b4827c475c6..008bf0bff186 100644 > > --- a/arch/powerpc/boot/Makefile > > +++ b/arch/powerpc/boot/Makefile > > @@ -453,8 +453,8 @@ clean-files += $(image-) $(initrd-) cuImage.* > > dtbImage.* treeImage.* \ > > clean-kernel-base := vmlinux.strip vmlinux.bin > > clean-kernel := $(addsuffix .gz,$(clean-kernel-base)) > > clean-kernel += $(addsuffix .xz,$(clean-kernel-base)) > > -# If not absolute clean-files are relative to $(obj). > > -clean-files += $(addprefix $(objtree)/, $(clean-kernel)) > > +# clean-files are relative to $(obj). > > +clean-files += $(addprefix ../../../, $(clean-kernel)) > > > > WRAPPER_OBJDIR := /usr/lib/kernel-wrapper > > WRAPPER_DTSDIR := /usr/lib/kernel-wrapper/dts > > diff --git a/scripts/Makefile.clean b/scripts/Makefile.clean > > index 74cb1c5c3658..878cec648959 100644 > > --- a/scripts/Makefile.clean > > +++ b/scripts/Makefile.clean > > @@ -36,13 +36,7 @@ __clean-files := \ > > > > __clean-files := $(filter-out $(no-clean-files), $(__clean-files)) > > > > -# clean-files is given relative to the current directory, unless it > > -# starts with $(objtree)/ (which means "./", so do not add "./" unless > > -# you want to delete a file from the toplevel object directory). > > - > > -__clean-files := $(wildcard > > \ > > -$(addprefix $(obj)/, $(filter-out $(objtree)/%, > > $(__clean-files))) \ > > -$(filter $(objtree)/%, $(__clean-files))) > > +__clean-files := $(wildcard $(addprefix $(obj)/, $(__clean-files))) > > > > # > > == > > > > -- > > 2.32.0 -- Best Regards Masahiro Yamada
Re: [PATCH] kbuild: drop $(objtree)/ prefix support for clean-files
Masahiro Yamada writes: > I think this hack is a bad idea. arch/powerpc/boot/Makefile is the > only user. Let's stop doing this. > > Signed-off-by: Masahiro Yamada > --- > > arch/powerpc/boot/Makefile | 4 ++-- Acked-by: Michael Ellerman (powerpc) cheers > scripts/Makefile.clean | 8 +--- > 2 files changed, 3 insertions(+), 9 deletions(-) > > diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile > index 4b4827c475c6..008bf0bff186 100644 > --- a/arch/powerpc/boot/Makefile > +++ b/arch/powerpc/boot/Makefile > @@ -453,8 +453,8 @@ clean-files += $(image-) $(initrd-) cuImage.* dtbImage.* > treeImage.* \ > clean-kernel-base := vmlinux.strip vmlinux.bin > clean-kernel := $(addsuffix .gz,$(clean-kernel-base)) > clean-kernel += $(addsuffix .xz,$(clean-kernel-base)) > -# If not absolute clean-files are relative to $(obj). > -clean-files += $(addprefix $(objtree)/, $(clean-kernel)) > +# clean-files are relative to $(obj). > +clean-files += $(addprefix ../../../, $(clean-kernel)) > > WRAPPER_OBJDIR := /usr/lib/kernel-wrapper > WRAPPER_DTSDIR := /usr/lib/kernel-wrapper/dts > diff --git a/scripts/Makefile.clean b/scripts/Makefile.clean > index 74cb1c5c3658..878cec648959 100644 > --- a/scripts/Makefile.clean > +++ b/scripts/Makefile.clean > @@ -36,13 +36,7 @@ __clean-files := \ > > __clean-files := $(filter-out $(no-clean-files), $(__clean-files)) > > -# clean-files is given relative to the current directory, unless it > -# starts with $(objtree)/ (which means "./", so do not add "./" unless > -# you want to delete a file from the toplevel object directory). > - > -__clean-files := $(wildcard \ > -$(addprefix $(obj)/, $(filter-out $(objtree)/%, > $(__clean-files))) \ > -$(filter $(objtree)/%, $(__clean-files))) > +__clean-files := $(wildcard $(addprefix $(obj)/, $(__clean-files))) > > # == > > -- > 2.32.0
[PATCH] kbuild: drop $(objtree)/ prefix support for clean-files
I think this hack is a bad idea. arch/powerpc/boot/Makefile is the only user. Let's stop doing this. Signed-off-by: Masahiro Yamada --- arch/powerpc/boot/Makefile | 4 ++-- scripts/Makefile.clean | 8 +--- 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile index 4b4827c475c6..008bf0bff186 100644 --- a/arch/powerpc/boot/Makefile +++ b/arch/powerpc/boot/Makefile @@ -453,8 +453,8 @@ clean-files += $(image-) $(initrd-) cuImage.* dtbImage.* treeImage.* \ clean-kernel-base := vmlinux.strip vmlinux.bin clean-kernel := $(addsuffix .gz,$(clean-kernel-base)) clean-kernel += $(addsuffix .xz,$(clean-kernel-base)) -# If not absolute clean-files are relative to $(obj). -clean-files += $(addprefix $(objtree)/, $(clean-kernel)) +# clean-files are relative to $(obj). +clean-files += $(addprefix ../../../, $(clean-kernel)) WRAPPER_OBJDIR := /usr/lib/kernel-wrapper WRAPPER_DTSDIR := /usr/lib/kernel-wrapper/dts diff --git a/scripts/Makefile.clean b/scripts/Makefile.clean index 74cb1c5c3658..878cec648959 100644 --- a/scripts/Makefile.clean +++ b/scripts/Makefile.clean @@ -36,13 +36,7 @@ __clean-files:= \ __clean-files := $(filter-out $(no-clean-files), $(__clean-files)) -# clean-files is given relative to the current directory, unless it -# starts with $(objtree)/ (which means "./", so do not add "./" unless -# you want to delete a file from the toplevel object directory). - -__clean-files := $(wildcard \ - $(addprefix $(obj)/, $(filter-out $(objtree)/%, $(__clean-files))) \ - $(filter $(objtree)/%, $(__clean-files))) +__clean-files := $(wildcard $(addprefix $(obj)/, $(__clean-files))) # == -- 2.32.0
