> Date: Fri, 18 Sep 2015 10:04:34 +0100 > From: Gavin Smith <gavinsmith0...@gmail.com> > Cc: Vincent Belaïche <vincent.belai...@gmail.com>, > Texinfo <bug-texinfo@gnu.org>, Karl Berry <k...@freefriends.org> > > On 17 September 2015 at 20:20, Eli Zaretskii <e...@gnu.org> wrote: > > In the beginning there was only 'test -n "$COMSPEC$ComSpec"', and the > > comment still says so. Then someone wanted to exclude Cygwin from > > that (since Cygwin is a Posix emulation, so it uses ':' even though > > $ComSpec is set, as always on Windows). But then someone added > > "mingw" and "djgpp" to that, which of course is a mistake, since they > > don't need the uname test to begin with. > > > > So just leave only cygwin there, and that's it. > > To confirm, you say that we should do the following? > > Index: texi2dvi > =================================================================== > --- texi2dvi (revision 6629) > +++ texi2dvi (working copy) > @@ -89,7 +89,7 @@ > # directories in TEXINPUTS -- except for Cygwin et al., where COMSPEC > # might be inherited, but : is used. > if test -n "$COMSPEC$ComSpec" \ > - && uname | $EGREP -iv 'cygwin|mingw|djgpp' >/dev/null; then > + && uname | $EGREP -iv 'cygwin' >/dev/null; then > path_sep=";" > else > path_sep=":"
Yes.
