On Monday 24 April 2023 17:08:31 Simon Glass wrote: > Add the required extension to the Makefile rule. > > Signed-off-by: Simon Glass <s...@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 >
