Created attachment 168448 [details]
Rename conflicting max_align_t definition
In bug 208158, I asked for an exp-run with updated libc++. It turns out that security/gnutls attempts to redefine max_align_t, which is also present in libc++'s copy of stddef.h, and this conflicts.
Since gnutls doesn't really use the type at all, I thought it was easiest to simply rename the type to gnu_max_align_t. This avoids any compilation errors of C++ sources in gnutls.
Alternatively, the whole block defining max_align_t could be deleted, but that is a more invasive change, and I don't know how the configure script will handle that...
Just append ac_cv_type_max_align_t=yes to CONFIGURE_ARGS in the port Makefile.
(In reply to Tijl Coosemans from comment #1)
> Just append ac_cv_type_max_align_t=yes to CONFIGURE_ARGS in the port
If that prevents gnutls from rolling its own copy, then it is even better, yes.
A commit references this bug:
Date: Mon Mar 21 12:03:40 UTC 2016
New revision: 411557
- Take maintainer.
- Fix a build problem with upcoming libc++.
PR: 207768, 208175
Approved by: bdrewery (maintainer)