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

network: don't enable gcrypt thread callbacks when gcrypt recent enough #793

Merged
merged 1 commit into from Nov 11, 2014

Conversation

vincentbernat
Copy link
Contributor

From gcrypt.h:

NOTE: Since Libgcrypt 1.6 the thread callbacks are not anymore used.
However we keep it to allow for some source code compatibility if used
in the standard way.

Otherwise, we get a deprecation warning which is turned into an error:

  CC       libcollectdclient_la-network_buffer.lo
../../../src/libcollectdclient/network_buffer.c:58:15: error: 'gcry_thread_cbs' is deprecated (declared at /usr/include/gcrypt.h:213) [-Werror=deprecated-declarations]
 GCRY_THREAD_OPTION_PTHREAD_IMPL;

Fixes: #632

From `gcrypt.h`:

> NOTE: Since Libgcrypt 1.6 the thread callbacks are not anymore used.
> However we keep it to allow for some source code compatibility if used
> in the standard way.

Otherwise, we get a deprecation warning which is turned into an error:

```
  CC       libcollectdclient_la-network_buffer.lo
../../../src/libcollectdclient/network_buffer.c:58:15: error: 'gcry_thread_cbs' is deprecated (declared at /usr/include/gcrypt.h:213) [-Werror=deprecated-declarations]
 GCRY_THREAD_OPTION_PTHREAD_IMPL;
```

Fixes: collectd#632
@mfournier mfournier added Bug A genuine bug build An issue with the build labels Nov 11, 2014
pyr added a commit that referenced this pull request Nov 11, 2014
network: don't enable gcrypt thread callbacks when gcrypt recent enough
@pyr pyr merged commit e51e0f2 into collectd:master Nov 11, 2014
@octo octo added Fix A pull request fixing a bug and removed Bug A genuine bug labels Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build An issue with the build Fix A pull request fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

collectd fails to build with -Werror because it uses deprecated gcrypt functions
4 participants