On Sun, Oct 23, 2022 at 12:58:34PM +0300, Eli Zaretskii wrote: > This could be due to Diff. I have long ago replaced the MSYS Diff > with the following shell script (and gladly forgot about this issue): > > #! /bin/sh > # diff --- like diff.exe but ignore trailing CR characters > /bin/real_diff --strip-trailing-cr $* > > where 'real_diff' is the original Diff program, renamed. > > So maybe the install-info test suite should do something similar.
We could use a shell variable DIFF in the tests and set DIFF='diff --strip-trailing-cr' when needed. This would keep the CR handling contained and stop it spreading throughout the code. There doesn't appear to be a similar option for 'grep'. However, since it is just the one test, we can easily work around the problem by taking the '$' out - the test in question would still work properly.
