On Fri, Jan 26, 2018 at 01:37:00PM +0100, SZEDER Gábor wrote: > The second 'test_i18ngrep' invocation in the test 'curl redirects > respect whitelist' is missing its filename parameter. This has > remained unnoticed since its introduction in f4113cac0 (http: limit > redirection to protocol-whitelist, 2015-09-22), because it would only > cause the test to fail if Git was built with a sufficiently old > libcurl version. The test's two ||-chained 'test_i18ngrep' > invocations are supposed to check that either one of the two patterns > is present in 'git clone's error message. As it happens, the first > invocation covers the error message from any reasonably up-to-date > libcurl, thus the second invocation, the one without the filename > parameter, isn't executed at all. Apparently no one has run the test > suite's httpd tests with such an old libcurl in the last 2+ years, or > at least they haven't bothered to notify us about the failed test. Interesting find. The "too old" curl is older than 7.19.4, which we actually fail to build with since v2.12.0. So they probably did not even get as far as the tests. ;) > Fix this by consolidating the two patterns into a single extended > regexp, eliminating the need for an ||-chained second 'test_i18ngrep' > invocation. OK. Once upon a time I think we had trouble with "grep -E", since some older systems had only "egrep". But I see we've introduced some "grep -E" invocations as far back as 2013 and nobody has complained, so it's probably fine. -Peff
