Re: [AUCTeX-devel] Re: preview-datadir and XEmacs

[David Kastrup]

Ralf Angeli <address@hidden> writes:

You are living one hour in the future.  Daylight saving time is not
that drastic.

> * David Kastrup (2006-03-21) writes:
>
>> Maybe.  If you want to try something, feel free to do so.
>
> The following patch fixes the bug for me.  The way preview/auto.el is
> being passed to `preview-make-package' (or not) might be considered
> peculiar.
>
> Index: Makefile.in
> ===================================================================
> RCS file: /cvsroot/auctex/auctex/Makefile.in,v
> retrieving revision 1.130
> diff -u -r1.130 Makefile.in
> --- Makefile.in       23 Jan 2006 13:48:40 -0000      1.130
> +++ Makefile.in       21 Mar 2006 19:23:16 -0000
> @@ -224,7 +224,7 @@
>  install-metadata:
>       test "x$(packagedir)" = xno || { \
>         $(MKINSTALLDIRS) $(DESTDIR)$(packagedir)/pkginfo || : ; \
> -       $(ELCC) -l preview/prv-install -f preview-make-package 
> $(DESTDIR)$(packagedir) "$(PACKAGE)" \"`echo $(AUCTEXDATE)|sed 
> 's/\(.*\)-\(.*\)-\(.*\)/\1.\2\3/'`\" \"$(AUCTEXVERSION)\" tex-site.el && \
> +       $(ELCC) -l preview/prv-install -f preview-make-package 
> $(DESTDIR)$(packagedir) "$(PACKAGE)" \"`echo $(AUCTEXDATE)|sed 
> 's/\(.*\)-\(.*\)-\(.*\)/\1.\2\3/'`\" \"$(AUCTEXVERSION)\" tex-site.el `{ test 
> "x$(preview_enabled)" != xyes || echo "preview/auto.el"; }` && \

That does not sound like a good idea since I would not bet on all
shells considering `true` to be nothing (namely, different from an
empty string argument).

Perhaps just use the duplication

        if test "x$(preview_enabled)" = yes; then \
           $(ELCC) ... tex-site.el preview/auto.el ; \
        else \
           $(ELCC) ... tex-site.el ; \
        fi && \

?

Sure, more wordy, but not much so.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum