Skip to content

/ curl

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test 1422 fails with --disable-file #2741

Closed
MarcelRaad opened this issue Jul 13, 2018 · 0 comments
Closed

Test 1422 fails with --disable-file #2741

MarcelRaad opened this issue Jul 13, 2018 · 0 comments
Assignees
@MarcelRaad
Labels
tests

Comments

@MarcelRaad
Copy link
Member

@MarcelRaad MarcelRaad commented Jul 13, 2018

I did this

./configure --enable-debug --enable-file && make && cd tests && make && ./runtests.pl 1422

test 1422...[HTTP GET with -O -J and Content-Disposition (empty file)]
-p---oe--- OK (1   out of 1  , remaining: 00:00)

./configure --enable-debug --disable-file && make && cd tests && make && ./runtests.pl 1422

test 1422...[HTTP GET with -O -J and Content-Disposition (empty file)]
curl returned 1, when expecting 0
 exit FAILED
 - abort tests

See https://curl.haxx.se/dev/log.cgi?id=20180713044637-30368#prob8:

=== Start of file trace1422
[...]
06:45:59.171000 == Info: Protocol "file" not supported or disabled in libcurl
[...]
=== End of file trace1422

I expected the following

The second test not to fail

curl/libcurl version

curl 7.60.0 (x86_64-pc-msys) libcurl/7.60.0 OpenSSL/1.0.2m zlib/1.2.11 brotli/1.0.5 libidn2/2.0.5 libpsl/0.19.1 (+libidn2/2.0.4) libssh2/1.8.0 nghttp2/1.30.0

operating system

MinGW-w64 on Windows
@MarcelRaad MarcelRaad added the tests label Jul 13, 2018
@MarcelRaad MarcelRaad self-assigned this Jul 13, 2018
MarcelRaad added a commit to MarcelRaad/curl that referenced this issue Jul 13, 2018
@MarcelRaad
test1422: add required file feature
Verified
This commit was signed with a verified signature.
MarcelRaad Marcel Raad
GPG key ID: 33C416EFAE4D6F02 Learn about signing commits
7019083 
curl configured with --enable-debug --disable-file currently complains
on test1422:
Info: Protocol "file" not supported or disabled in libcurl

Make test1422 dependend on enabled FILE protocol to fix this.

Fixes curl#2741
Closes
MarcelRaad added a commit to MarcelRaad/curl that referenced this issue Jul 13, 2018
@MarcelRaad
test1422: add required file feature
Verified
This commit was signed with a verified signature.
MarcelRaad Marcel Raad
GPG key ID: 33C416EFAE4D6F02 Learn about signing commits
df7631b 
curl configured with --enable-debug --disable-file currently complains
on test1422:
Info: Protocol "file" not supported or disabled in libcurl

Make test1422 dependend on enabled FILE protocol to fix this.

Fixes curl#2741
Closes
@MarcelRaad MarcelRaad mentioned this issue Jul 13, 2018
Closed
falconindy added a commit to falconindy/curl that referenced this issue Sep 10, 2018
@MarcelRaad @falconindy
test1422: add required file feature
048eed9 
curl configured with --enable-debug --disable-file currently complains
on test1422:
Info: Protocol "file" not supported or disabled in libcurl

Make test1422 dependend on enabled FILE protocol to fix this.

Fixes curl#2741
Closes curl#2742
@lock lock bot locked as resolved and limited conversation to collaborators Oct 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MarcelRaad MarcelRaad

Labels
tests
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

test1422: add required file feature
1 participant
@MarcelRaad
You can’t perform that action at this time.