Created attachment 191621 [details] Patch php56-mbstring blithely assumes that the implementation of Oniguruma's regex_t is public. Since release 6.8.1, it is not. I have already submitted the same patch as PHP bug 76113; https://bugs.php.net/bug.php?id=76113 .
Thank you Christian
php71_mbstring fails also /ram/usr/ports/converters/php71-mbstring/work/php-7.1.15/ext/mbstring/php_mbregex.c:455:15: error: incomplete definition of type 'struct php_mb_re_pattern_buffer' if (!rc || rc->options != options || rc->enc != enc || rc->syntax != syntax) { ~~^ /usr/local/include/oniguruma.h:673:8: note: forward declaration of 'struct php_mb_re_pattern_buffer' struct re_pattern_buffer; ^ ./php_onig_compat.h:4:37: note: expanded from macro 're_pattern_buffer' #define re_pattern_buffer php_mb_re_pattern_buffer ^ the needed patch is different from the patch of php56-mbstring.
*** Bug 226728 has been marked as a duplicate of this bug. ***
Created attachment 191629 [details] php 7.x patch Congratulations for identifyiing the simple patch! I'm using php72 and the same failure happens. PHP 7.x has a slightly different logic in the failing point. I've added a patch for PHP 7.2(based on the yours), it should be the same for 7.0 nd 7.1. It's working fine for me.
I've taken the freedom to correct the title, since the bug is affecting all PHP versions.
Thanks for the patches. I will have a look at them!
A commit references this bug: Author: antoine Date: Mon Mar 19 23:01:51 UTC 2018 New revision: 465040 URL: https://svnweb.freebsd.org/changeset/ports/465040 Log: Fix build with oniguruma 6.8.1 PR: 226717 Submitted by: Christian Ullrich, madpilot With hat: portmgr Changes: head/converters/php56-mbstring/files/patch-php__mbregex.c head/converters/php70-mbstring/files/patch-php__mbregex.c head/converters/php71-mbstring/files/patch-php__mbregex.c head/converters/php72-mbstring/files/patch-php__mbregex.c