Hello Robert, * Robert Millan wrote on Fri, Jul 24, 2009 at 08:36:59PM CEST: > > This patch makes it possible to use autotest on MingW32/Wine, where > DOS carriage returns are printed, breaking the testsuite (for the > record, problem was found with GNU cpio's testsuite).
In addition to Eric's review, > +# Use `diff --strip-trailing-cr' when possible. > +if at_diff_tmp=`$at_diff -u "$at_devnull" "$at_devnull" 2>&1` && test -z > "$at_diff_tmp" this test does not really ensure that --strip-trailing-cr is accepted by the diff program. So if you add another macro that compares only text files (say AT_CHECK_TEXT_FILE or so), ignoring line endings, it should test this option. If the option is not supported, then another means should be used to normalize the files for the test, e.g., by using tr, so that the functionality can be provided portably. For example, Libtool's new testsuite does something similar within its LT_AT_UNIFY_NL macro. > +then > + at_diff="$at_diff --strip-trailing-cr" > fi > > # Get the last needed group. Cheers, Ralf
