RE: [PATCH] ARM: simplify the build rule of mach-types.h

From: David Laight
Date: Mon May 31 2021 - 05:00:23 EST


From: Masahiro Yamada
> Sent: 28 May 2021 04:49
>
> The directory of mach-types.h is created a couple of lines above:
>
> _dummy := $(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)') \
>
> The 'mkdir -p' command is redundant.
(In the rule itself)
...
> quiet_cmd_gen_mach = GEN $@
> - cmd_gen_mach = mkdir -p $(dir $@) && \
> - $(AWK) -f $(filter-out $(PHONY),$^) > $@
> + cmd_gen_mach = $(AWK) -f $(real-prereqs) > $@

There is a much easier way to get directories created.
gmake lets you define dependencies that only need to exist
(ie there file timestamps are't checked)
These are ideal for creating directories.

So if can define:

%/.:
mkdir -p $@

You can just use

$(OBJ)/$(FILE): | ${OBJ)/.

to get any directories created.

Annoyingly gmake doesn't seem to support the 'dynamic dependencies'
of SYSV make (it's only useful feature).
So you can't use:
xxxxx: | $$@D/.

David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
Registration No: 1397386 (Wales)