-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
pcre2: Update to 10.43 #89371
pcre2: Update to 10.43 #89371
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Could you squash the commits?
Thanks! |
I'm running into a build issue for iOS (with osxcross) on our official buildsystem:
|
I have no clue what I'm doing, but this seems to work it around: diff --git a/thirdparty/pcre2/src/sljit/allocator_src/sljitExecAllocatorApple.c b/thirdparty/pcre2/src/sljit/allocator_src/sljitExecAllocatorApple.c
index 95b9842fa9..3fc8091ead 100644
--- a/thirdparty/pcre2/src/sljit/allocator_src/sljitExecAllocatorApple.c
+++ b/thirdparty/pcre2/src/sljit/allocator_src/sljitExecAllocatorApple.c
@@ -104,6 +104,7 @@ static SLJIT_INLINE void apple_update_wx_flags(sljit_s32 enable_exec)
#else
#define SLJIT_MAP_JIT (0)
#endif
+#define SLJIT_UPDATE_WX_FLAGS(from, to, enable_exec)
#endif /* TARGET_OS_OSX */
|
I think we're also compiling too many files, |
Yeah, that might be on me 😬 Whatever's going on with |
https://github.com/PCRE2Project/pcre2/releases/tag/pcre2-10.43
thirdparty/README.md
as it seemed to omit mentioning one file