On 26.10.2020 21:41, Olaf Hering wrote: > Use a temporay file, and move it in place once done. > The same pattern exists already for other dependencies.
This pattern is used when a rule consists of multiple commands having their output appended to one another's. This isn't the case here, so I'm afraid I'd like it to be made more obvious what the gain / improvement is. That is - I don't mind the change, if indeed it is for a good reason. Jan > Signed-off-by: Olaf Hering <o...@aepfle.de> > --- > tools/libacpi/Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools/libacpi/Makefile b/tools/libacpi/Makefile > index c17f3924cc..2cc4cc585b 100644 > --- a/tools/libacpi/Makefile > +++ b/tools/libacpi/Makefile > @@ -43,7 +43,8 @@ all: $(C_SRC) $(H_SRC) > > $(H_SRC): $(ACPI_BUILD_DIR)/%.h: %.asl iasl > iasl -vs -p $(ACPI_BUILD_DIR)/$*.$(TMP_SUFFIX) -tc $< > - sed -e 's/AmlCode/$*/g' -e 's/_aml_code//g' $(ACPI_BUILD_DIR)/$*.hex >$@ > + sed -e 's/AmlCode/$*/g' -e 's/_aml_code//g' $(ACPI_BUILD_DIR)/$*.hex > >$@.$(TMP_SUFFIX) > + mv -f $@.$(TMP_SUFFIX) $@ > rm -f $(addprefix $(ACPI_BUILD_DIR)/, $*.aml $*.hex) > > $(MK_DSDT): mk_dsdt.c >
