archivers/brotli: Update to 0.6.0 - Update brotli to 0.6.0 - Move from devel/libbrotli - No longer requires the meta project - Switch from autoconf to cmake - Fix nginx brotli module PR: 218851 Submitted by: Markus Kohlmeyer <email@example.com> Reviewed_by: Approved by: Differential_Revision
- rP445604: archivers/brotli: Move and update to 0.6.0
@mat See inline comment
I think this is a problem of brotli itself - it's not possible to have both types of libraries. See:
Agree. This was on request of osa@ (see PR). Gone now.
The only other dependency I've found is in mail/cyrus-imapd30 where it is commented out, with the new archivers/brotli this builds fine
-# Our libbrotli is too old -#HTTP_LIB_DEPENDS+= libbrotlidec.so:devel/libbrotli +HTTP_LIB_DEPENDS+= libbrotlicommon.so:archivers/brotli
Also, is there any need for the static stuff ? Using static libraries is a pain in the ports tree. Also ELF has been here for 20 years, nobody should be using or providing static libs, ever.
This should be ?=, and also only brotli-static.
So one can ask the brotli port to build as STATIC, this is not really great. Maybe simply drop options and use a .if defined(STATIC_ONLY) / .else