[PATCH 26/31] Makefile: Correct the ans1_compiler rule for MSYS2
Pali Rohár
pali at kernel.org
Tue Apr 25 18:23:16 CEST 2023
On Monday 24 April 2023 17:08:31 Simon Glass wrote:
> Add the required extension to the Makefile rule.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Makefile | 1 +
> scripts/Makefile.build | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 773260b81d5d..08eab87d0a11 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -48,6 +48,7 @@ ifeq ($(MSYS_VERSION),0)
> export SOEXT := so
> else
> export SOEXT := dll
> +export ELFEXT := .exe
.EXE is not extension for ELF format :-)
Autotools and also other tools use EXEEXT Makefile variable for this.
What about following this de-factor standard variable name?
> endif
>
> # Avoid funny character set dependencies
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index 97dd4a64f6ef..547dc039ceb4 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -309,7 +309,7 @@ quiet_cmd_asn1_compiler = ASN.1 $@
> cmd_asn1_compiler = $(objtree)/tools/asn1_compiler $< \
> $(subst .h,.c,$@) $(subst .c,.h,$@)
>
> -$(obj)/%.asn1.c $(obj)/%.asn1.h: $(src)/%.asn1 $(objtree)/tools/asn1_compiler
> +$(obj)/%.asn1.c $(obj)/%.asn1.h: $(src)/%.asn1 $(objtree)/tools/asn1_compiler$(ELFEXT)
> $(call cmd,asn1_compiler)
>
> # Build the compiled-in targets
> --
> 2.40.0.634.g4ca3ef3211-goog
>
More information about the U-Boot
mailing list