-
Notifications
You must be signed in to change notification settings - Fork 80
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
hyperscan: replacing wrapper #475
Conversation
Nix issues are because poetry2nix not resolving dependencies from custom registry sources. I'll publish the package on pypi, this will clear the remaining errors. |
Works like a charm ! |
Currently doing a bunch of tests on ARM based platforms. |
I am planning to release aarch64 wheels also |
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.
Read through the pyperscan repo and this MR. I also checked that unblob tests passed on amd64 and aarch64 with this change.
Note: to fix #415 you'll need to publish wheels for aarch64 Linux and OSX. |
I have wheels for OSX, only aarch64 linux wheels are missing. |
This new wrapper is written by me and comes with a statically linked vectorscan library. Extensive testing and a proper pypi release is needed before this can be merged.
Hopefully this can fix #415 and reduce friction endured during installation like in #467.
Also would help packaging for Nixpkgs (#424) and other distros, as dynamic linking to a system provided hyperscan library is also a possibility.
Currently the only dependency required is C++ runtime (
libstdc++
and its mac counterpart) which is much easier to come by.