Re: AUCTeX doesn't build with latest Emacs master

[Arash Esbati]

Hi Keita,

Ikumi Keita <ikumi@ikumi.que.jp> writes:

>>>>>> Arash Esbati <arash@gnu.org> writes:
>> My suggestion is now apply the following change:
>
> Thank you for looking into this, but it seems that the commit 19a2607b
> broke compatibility.
>
> @@ -257,6 +268,9 @@ tex-site.el: tex-site.el.out auto-loads.el Makefile
>       cat auto-loads.el >>$@
>       echo "(provide 'tex-site)" >>$@ ; \
>       echo ";;; tex-site.el ends here" >>$@
> +     sed -i -e "/^(provide 'auto-loads)/d" \
> +            -e '/^;;; auto-loads.el ends here/d' \
> +            -e 's/^\(;;; auto-loads.el.*\)\(   -\*- lexical-binding: t 
> -\*-\)/\1/' $@
>  
>  tex-site.el.out: tex-site.el.in Makefile config.status
>       ./config.status
>
> This change causes the following error, at least for FreeBSD 13.1. This
> is a quote from the output of "make":
> ----------------------------------------------------------------------
> sed -i -e "/^(provide 'auto-loads)/d" \
>        -e '/^;;; auto-loads.el ends here/d' \
>        -e 's/^\(;;; auto-loads.el.*\)\(   -\*- lexical-binding: t -\*-\)/\1/' 
> tex-site.el
> sed: -e: No such file or directory
> make: *** [Makefile:270: tex-site.el] Error 1
> ----------------------------------------------------------------------
> It seems that "extension" argument is mandatory for "-i" option of
> FreeBSD sed. Here is the quote from output of "man sed":

Thanks for catching this.  I admit that I tried it only with GNU sed.
Can you please apply this small change to your tree and tell if it
works?  TIA.

--8<---------------cut here---------------start------------->8---
diff --git a/Makefile.in b/Makefile.in
index 9d97eff3..97ddae21 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -268,9 +268,11 @@ tex-site.el: tex-site.el.out auto-loads.el Makefile
        cat auto-loads.el >>$@
        echo "(provide 'tex-site)" >>$@ ; \
        echo ";;; tex-site.el ends here" >>$@
-       sed -i -e "/^(provide 'auto-loads)/d" \
-              -e '/^;;; auto-loads.el ends here/d' \
-              -e 's/^\(;;; auto-loads.el.*\)\(   -\*- lexical-binding: t 
-\*-\)/\1/' $@
+       sed -i'.tmp' \
+           -e "/^(provide 'auto-loads)/d" \
+           -e '/^;;; auto-loads.el ends here/d' \
+           -e 's/^\(;;; auto-loads.el.*\)\(   -\*- lexical-binding: t 
-\*-\)/\1/' $@
+       -rm -f $@.tmp

 tex-site.el.out: tex-site.el.in Makefile config.status
        ./config.status
--8<---------------cut here---------------end--------------->8---

Best, Arash