[PATCH v3 2/3] emacs: Add simple make target to compile emacs lisp tests

Jonas Bernoulli jonas at bernoul.li
Tue May 26 12:49:00 PDT 2020


David Bremner <david at tethera.net> writes:

> Jonas Bernoulli <jonas at bernoul.li> writes:
>
>> +compile-elisp-tests:
>> +	$(EMACS) --batch -L emacs -L test -l notmuch.el -l test-lib.el -f \
>> +	batch-byte-compile test/*.el
>> +
>
> Can you explain a bit (perhaps in an updated commit message) why we need
> this target?

The tests might have issues that can be detected by the byte-compiler.
The only issue that I actually did detect was the one fixed by 3/3,
which is a very minor cosmetic issue.

However if someone modified the tests, then they might of course add
more serious issues.  If they ran this target, then they might detect
that.  But they might forget to do so.  They might also forget to re-run
this target after making changes.

Therefore I retract this commit at least in the current form. I might
include an improved more thoroughly described version of this in a later
patch series.  For now please just apply 3/3.

     Jonas


More information about the notmuch mailing list