bug#41329: [feature/native-comp] The Makefile only checks for .elc files.

[Andrea Corallo]

Nicolas Bértolo <nicolasbertolo@gmail.com> writes:

>> I'm not sure I understand.  I you mean that manually byte-compiling
> a
>> file prevents the .eln compilation to happen if make is invoked
>> afterwards then yes.  But make should be used.
>
> I use `auto-compile-mode` so that is why I was hitting this problem.

I see, I rarely messed up externally the repo and always used make to
build.

>> I don't think so, the .elc is renamed with their final filename
> after
>> the .eln has been produced to prevent exactly that.
>
> Mmm, it has happened in my system. I wonder why. Will post an update
> if I manage to reproduce it reliably.

I suspect this is due to some confusion or a different issue to the
discussed one.

AFAIU if a file is compiled by `batch-byte-native-compile-for-bootstrap'
the elc can be produced only after the eln.

>> Yes the main reason for that is that the Makefile is not aware of
> which
>> files are lexical and can be compiled and which are dynamic and
> can't.
>
> Can't it grep for "lexical-binding: t" in the first line?

I think so, but playing with the build system can be time consuming,
especially to obtain a portable solutions.  Given that when we'll have
dynamic scope compilation will be sufficient just not to produce the
.elc I decided not to invest more time on this.

If you want to suggest a patch to have the eln files as target this is
very welcome.  Consider that the other complication is the eln folder
with the hash that should be know to the build system, so probably a two
step solution is required.

Thanks

  Andrea

--
akrl@sdf.org