Re: 1.4 and AUX_DIR and Texinfo

[Tim Van Holder]

>  .texi.dvi:
> -       TEXINPUTS=.:$$TEXINPUTS \
> +       TEXINPUTS=$(top_srcdir)/config/texinfo.tex:$$TEXINPUTS \
>           MAKEINFO='$(MAKEINFO) -I $(srcdir)' $(TEXI2DVI) $<
> 
> This is wrong, it should read
> 
>  .texi.dvi:
> -       TEXINPUTS=.:$$TEXINPUTS \
> +       TEXINPUTS=$(top_srcdir)/config:$$TEXINPUTS \
>           MAKEINFO='$(MAKEINFO) -I $(srcdir)' $(TEXI2DVI) $<
> 
<DJGPP advocacy mode>
I'd like to add that you may want to
 a) use (double) quotes around $(top_srcdir)/config:$$TEXINPUTS,
    as top_srcdir may contain spaces (especially on Windows)
    There's probably a few other places where this needs to be
    considered.
 b) use @PATH_SEPARATOR@ instead of ':'; this variable is new to
    autoconf 2.50 though, so you may need to add some code to
    init.m4 that duplicates autoconf's pathsep test if
    $ac_path_separator isn't set (autoconf now sets this during
    AC_INIT).  This is one of the few places in automake where
    an unportable construct is used.  I've had it patched in my
    local devel version for quite a while now, but now autoconf
    2.50 (with the pathsep test) is out, I figure it's time to
    get this right in the official tree.
</DJGPP advocacy mode>