jyapayne/swig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[php] Workaround PHP 8.2 header problem

Browse source 
We need to include php.h before stdio.h (and probably before most
other libc headers).
This commit is contained in:
Olly Betts 2022-09-28 19:01:01 +13:00
commit 246dd2d787
2 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Source/Modules/php.cxx
View file

@ -545,6 +545,14 @@ public:
Swig_banner(f_begin);
// We need to include php.h before string.h gets included, at least with
// PHP 8.2. Otherwise string.h is included without _GNU_SOURCE being
// included and memrchr() doesn't get declared, and then inline code in
// the PHP headers defines _GNU_SOURCE, includes string.h (which is a
// no op thanks to the include gaurds), then tries to use memrchr() and
// fails.
Append(f_runtime, "#include \"php.h\"\n\n");
Printf(f_runtime, "\n\n#ifndef SWIGPHP\n#define SWIGPHP\n#endif\n\n");
if (directorsEnabled()) {