bug-texinfo
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: diff wrapper for install-info tests


From: Bruno Haible
Subject: Re: diff wrapper for install-info tests
Date: Sun, 23 Oct 2022 16:06:42 +0200

Gavin Smith wrote:
> Could you try a change like the one at the end of this mail?
> 
> If that works, then we can make this change throughout the install-info
> test suite.

With the attached patch, all ii-????-test tests pass on mingw.

> The test for mingw may need tweaking to include/exclude MinGW, MSYS,
> cygwin...

Yes, it should test the host platform, not the build platform. I used
  echo '@host_os@'
and config.status replaces that with
  echo 'mingw32'

Also I removed the 'v' option from the $EGREP command (since we want the
$EGREP command to succeed, not to fail, in the mingw case), and removed
the 'i' option as well (since all config host values are lowercase ASCII
anyway).

> The --strip-trailing-cr option is specific to GNU diff (not in POSIX) but
> it may be ok for mingw.

Yes, the known environments for compiling mingw binaries (mingw, msys2,
cygwin, debian) all have GNU diff in $PATH.

Attachment: mingw.diff
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]