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

Janux 1.x commit d505788 fails to build on Fedora Rawhide #3131

Closed
bkmgit opened this issue Dec 27, 2022 · 2 comments
Closed

Janux 1.x commit d505788 fails to build on Fedora Rawhide #3131

bkmgit opened this issue Dec 27, 2022 · 2 comments
Labels
multistream Related to Janus 1.x

Comments

@bkmgit
Copy link
Contributor

bkmgit commented Dec 27, 2022

What version of Janus is this happening on?
Commit d505788
and release 1.0.4
Have you tested a more recent version of Janus too?
This is current development head

Was this working before?
Was working with other compilers

Is there a gdb or libasan trace of the issue?
Not applicable.

Additional context
Add any other context about the problem here.
With Gcc 12.2.1 20221121 get
In file included from /usr/include/curl/curl.h:3195,
from turnrest.c:21:
turnrest.c: In function ‘janus_turnrest_request’:
turnrest.c:168:9: error: void value not ignored as it ought to be
168 | curl_easy_setopt(curl, api_http_get ? CURLOPT_HTTPGET : CURLOPT_POST, 1);
| ^

Checking version of curl gives
$ curl --version
curl 7.87.0 (aarch64-redhat-linux-gnu) libcurl/7.87.0 OpenSSL/3.0.5 zlib/1.2.13 brotli/1.0.9 libidn2/2.3.4 libpsl/0.21.1 (+libidn2/2.3.3) libssh/0.10.4/openssl/zlib nghttp2/1.51.0
Release-Date: 2022-12-21
Protocols: dict file ftp ftps gopher gophers http https imap imaps ldap ldaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: alt-svc AsynchDNS brotli GSS-API HSTS HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM NTLM_WB PSL SPNEGO SSL threadsafe TLS-SRP UnixSockets

@bkmgit bkmgit added the multistream Related to Janus 1.x label Dec 27, 2022
@jonassmedegaard
Copy link

Same failure now occurs on Debian unstable as well, as reported here: https://bugs.debian.org/1027656

@lminiero
Copy link
Member

lminiero commented Jan 4, 2023

Closing. thanks for the fix!

@lminiero lminiero closed this as completed Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multistream Related to Janus 1.x
Projects
None yet
Development

No branches or pull requests

3 participants