If the input file didn't change, there is no point in repeating the file size check and cluttering the log. Therefore use if_changed and remove the extraneous semicolon, which make would then complain about. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- images/Makefile | 2 +- scripts/Makefile.lib | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/images/Makefile b/images/Makefile index 9007911278e2..80ea582728a5 100644 --- a/images/Makefile +++ b/images/Makefile @@ -78,7 +78,7 @@ $(obj)/%.pblb: $(obj)/%.pbl FORCE $(call if_changed,objcopy_bin,$(*F)) $(Q)$(OBJCOPY) -O binary --only-section=.missing_fw $< $@.missing-firmware $(Q)[ -s $@.missing-firmware ] || rm -f $@.missing-firmware - $(call cmd,check_file_size,$@,$(CONFIG_BAREBOX_MAX_IMAGE_SIZE)) + $(call if_changed,check_file_size,$@,$(CONFIG_BAREBOX_MAX_IMAGE_SIZE)) # # For each start symbol create three variables containing the diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index e8a85f3f1bf6..b698161f43f6 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -531,7 +531,7 @@ quiet_cmd_check_file_size = CHKFILESIZE $2 then \ echo "$@ size $$size > maximum size $$max_size" >&2; \ exit 1 ; \ - fi; + fi quiet_cmd_imximage__S_dcd= DCD_S $@ cmd_imximage_S_dcd= \ -- 2.39.5
