On Fri, May 22, 2020 at 11:00 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > I do not see a good reason to add ifdef here. > > Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx> > --- Applied to linux-kbuild. > scripts/Makefile.build | 10 ++-------- > 1 file changed, 2 insertions(+), 8 deletions(-) > > diff --git a/scripts/Makefile.build b/scripts/Makefile.build > index 3665b1a0bc8e..9af88f4cacb8 100644 > --- a/scripts/Makefile.build > +++ b/scripts/Makefile.build > @@ -384,16 +384,14 @@ $(obj)/%/built-in.a: $(obj)/% ; > # > # Rule to compile a set of .o files into one .a file (without symbol table) > # > -ifdef builtin-target > > quiet_cmd_ar_builtin = AR $@ > cmd_ar_builtin = rm -f $@; $(AR) cDPrST $@ $(real-prereqs) > > -$(builtin-target): $(real-obj-y) FORCE > +$(obj)/built-in.a: $(real-obj-y) FORCE > $(call if_changed,ar_builtin) > > targets += $(builtin-target) > -endif # builtin-target > > # > # Rule to create modules.order file > @@ -408,15 +406,11 @@ $(modorder-target): $(subdir-ym) FORCE > # > # Rule to compile a set of .o files into one .a file (with symbol table) > # > -ifdef lib-target > - > -$(lib-target): $(lib-y) FORCE > +$(obj)/lib.a: $(lib-y) FORCE > $(call if_changed,ar) > > targets += $(lib-target) > > -endif > - > # NOTE: > # Do not replace $(filter %.o,^) with $(real-prereqs). When a single object > # module is turned into a multi object module, $^ will contain header file > -- > 2.25.1 > -- Best Regards Masahiro Yamada
