Re: byte compilation in older emacs [patch]

On Fri, Jan 3, 2020 at 6:13 PM Karl Berry <[email protected]> wrote:
>
> Sigh. I failed to note that vc-dwim did not dwim; my summary first line
> was once again not recognized. (And I guess I should update the date.)
> I'll figure that out before pushing, but anyway, the content of the
> patch is what really needs review. Here's the actual ChangeLog entry as
> written ... -k
>
> 2020-01-03  Karl Berry  <[email protected]>
>
>         automake: Support byte compilation in older Emacsen
>
>         * lib/am/lisp.am (am__emacs_byte_compile_setup) [FIRST]: define
>         new make variable, to use byte-compile-dest-file-function if
>         available, else byte-compile-dest-file.
>         (.el.elc): use it.
>         * t/lisp-loadpath.sh: skip test if emacs version is <= 23,
>         since their -L ordering is backwards.
>         * NEWS: update.

Thank you for doing that. It looks fine to me. One request:

In this:
+emacs_major=$(${EMACS-emacs} --version | sed -e 's/.* //' -e 's/\..*$//' -e 1q)

please use this same-functionality sed command. I prefer to eliminate
such uses of -e and to reduce quote count:
  sed 's/.* //;s/\..*$//;1q'