Re: [PATCH 3/4] kbuild: create object directories simpler and faster

From: Masahiro Yamada
Date: Fri Nov 10 2017 - 04:22:17 EST


Hi Cao,


2017-11-10 17:45 GMT+09:00 Cao jin <caoj.fnst@xxxxxxxxxxxxxx>:

>> +ifneq ($(KBUILD_SRC),)
>> +# Create directories for object files if directory does not exist
>> +obj-dirs := $(sort $(obj) $(patsubst %/,%, $(dir $(targets))))
>> +$(shell mkdir -p $(obj-dirs))
>> +endif
>> +
>
> I just take a quick glance: is "$(obj)" here necessary? I think all
> $(targets) are under directory $(obj) when we descend into $(obj) to
> recursive make, if I don't miss anything.

Good catch!

I agree that $(obj) is unnecessary.
I will remove it if I see no problem in testing.

Thanks!


--
Best Regards
Masahiro Yamada