Skip to content
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

Disabling zstd results in empty state in configure output #5720

Closed
mback2k opened this issue Jul 24, 2020 · 3 comments
Closed

Disabling zstd results in empty state in configure output #5720

mback2k opened this issue Jul 24, 2020 · 3 comments
Assignees
@bagder
Labels
build

Comments

@mback2k
Copy link
Member

mback2k commented Jul 24, 2020

I did this

./configure --without-zstd

  curl version:     7.72.0-DEV
  SSL:              enabled (Windows-native, OpenSSL)
  SSH:              enabled (libSSH2)
  zlib:             no      (--with-zlib)
  brotli:           no      (--with-brotli)
  zstd:
  GSS-API:          no      (--with-gssapi)
  TLS-SRP:          enabled
  resolver:         POSIX threaded
  IPv6:             enabled
  Unix sockets:     no      (--enable-unix-sockets)
  IDN:              enabled (Windows-native)
  Build libcurl:    Shared=yes, Static=yes
  Built-in manual:  no      (--enable-manual)
  --libcurl option: enabled (--disable-libcurl-option)
  Verbose errors:   enabled (--disable-verbose)
  Code coverage:    disabled
  SSPI:             enabled
  ca cert bundle:   no
  ca cert path:     no
  ca fallback:      no
...

I expected the following

I expected a "no" behind "zstd:" instead of nothing.

curl/libcurl version

current master
@mback2k
Copy link
Member Author

mback2k commented Jul 24, 2020

@gvollant please take a look.

@gvollant
Copy link
Contributor

pehaps anything like

gillesvollant@gv-w81x64:/mnt/n/devN$ diff /mnt/n/forkcurl/configure.ac /mnt/s/curl/configure.ac
149a150

curl_zstd_msg="no (--with-zstd)"
`

*** /mnt/n/forkcurl/configure.ac 2020-07-12 18:16:50.798299000 +0200
--- /mnt/s/curl/configure.ac 2020-07-25 02:06:04.142077000 +0200

*** 147,152 ****
--- 147,153 ----
curl_ssh_msg="no (--with-{libssh,libssh2})"
curl_zlib_msg="no (--with-zlib)"
curl_brotli_msg="no (--with-brotli)"

  • curl_zstd_msg="no (--with-zstd)"
    curl_gss_msg="no (--with-gssapi)"
    curl_tls_srp_msg="no (--enable-tls-srp)"
    curl_res_msg="default (--enable-ares / --enable-threaded-resolver)"

@bagder bagder added the build label Jul 25, 2020
@bagder
Copy link
Member

bagder commented Jul 25, 2020

@gvollant yes, seems accurate!

bagder added a commit that referenced this issue Jul 27, 2020
@bagder
configure: show zstd "no" in summary when built without it
fb80f27 
Reported-by: Marc Hörsken
Fixes #5720
@bagder bagder mentioned this issue Jul 27, 2020
Closed
@bagder bagder self-assigned this Jul 27, 2020
@bagder bagder closed this as completed in ba39022 Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bagder bagder

Labels
build
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

configure: show zstd "no" in summary when built without it curl/curl
3 participants
@bagder @mback2k @gvollant