-
Notifications
You must be signed in to change notification settings - Fork 18
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
problem with non-JIT pcre2 #100
Comments
Hi @0-wiz-0 , |
This also happens with real regex, like:
|
Oh, wait:
No JIT is almost certainly the problem. Does NetBSD have a JIT-enabled PCRE2 available via its package manager or some other reasonable means? The question of whether to support a no-JIT PCRE/PCRE2 or not has been nagging at me for some time. It's part of the secret sauce that makes |
I've just built pcre2 with JIT support, and ucg now works fine both with strings and actual regexp. One other issue: on a PaX MPROTECT system, ucg built this way needs to be marked as not MPROTECT safe. On NetBSD this is done by calling 'paxctl +m ucg' after the build is finished. (see http://netbsd.gw.com/cgi-bin/man-cgi?paxctl++NetBSD-current) |
Hi again @0-wiz-0 ,
Indeed. I think I've decided that I should require a JIT-enabled PCRE/PCRE2 at configure-time, which would eliminate this as an error.
Yeah, I've been wondering when I was going to run into issues with JIT vs. the NX bit. Thanks for the info; I'll file a separate issue for that. |
…kle/ucg into bug/ISSUE100-NonJITPCRE2 * 'bug/ISSUE100-NonJITPCRE2' of https://github.com/gvansickle/ucg: Updated NEWS.md for issue #100.
* master: (44 commits) Updated NEWS.md for issue #100. Comment on OSX 10.11. Removed OSX 10.11 from rotation due to stty issues from #103. Put stty raw back for linux-style script. Trying to get travis to pull down a pcre2 for trusty. Travis change to add pcre2 to linux builds. Travis. Interim. Interim. Interim. Interim. Interim. Interim. Interim. Interim. Interim. Interim. testsuite, capturing intermediate files. script workaround for some OSX distro versions. Interim. printf portability fix. ... # Conflicts: # src/libext/filesystem.hpp
ucg
compiled with pcre seems to work fine.When compiled with pcre2-10.22, it builds fine but fails for me at runtime with:
Not sure what this is about (and there's a weird '/' at the start of the line).
ucg --version
output:pcre2's self tests run fine.
The text was updated successfully, but these errors were encountered: