Re: [PATCH] t5562: skip if NO_CURL is enabled
- Date: Mon, 19 Nov 2018 11:36:08 -0800
- From: Carlo Arenas <carenas@xxxxxxxxx>
- Subject: Re: [PATCH] t5562: skip if NO_CURL is enabled
On Mon, Nov 19, 2018 at 10:40 AM Max Kirillov <max@xxxxxxxxxx> wrote:
> On Mon, Nov 19, 2018 at 02:15:35AM -0800, Carlo Marcelo Arenas Belón wrote:
> > 6c213e863a ("http-backend: respect CONTENT_LENGTH for receive-pack", 2018-07-27)
> > introduced all tests but without a check for CURL support from git.
> The tests should not be using curl, they just pipe data to
> http-backend's standard input.
NO_CURL reflects the build setting (for http support); CURL checks for
the curl binary, but as Ævar points out the requirements must be from
somewhere else since a NO_CURL=1 build (tested in macOS) still passes
the test, but not in NetBSD.
tests 3-8 seem to fail because perl is hardcoded to /urs/bin/perl in
t5562/invoke-with-content-length.pl, while I seem to be getting some
sporadic errors in 9 with the following output :
++ env CONTENT_TYPE=application/x-git-receive-pack-request
++ verify_http_result '200 OK'
++ grep fatal: act.err
Binary file act.err matches
++ return 1
error: last command exited with $?=1
not ok 9 - push plain
and the following output in act.err (with a 200 in act)
fatal: the remote end hung up unexpectedly