Skip to content
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

Switch from Hyperscan to a bundled version of Vectorscan #41

Merged
merged 30 commits into from
Mar 31, 2023

Conversation

bradlarsen
Copy link
Collaborator

@bradlarsen bradlarsen commented Mar 28, 2023

This pull request switches from using the rust-hyperscan crate to using a custom set of bindings based on those found in pyperscan. These new custom bindings statically link against a bundled version of Vectorscan that is built from source, instead of linking sometimes non-statically against Hyperscan.

These new custom bindings appear in the vectorscan and vectorscan-sys directories.

These changes make it simpler to build Nosey Parker from source, especially on ARM systems, and make it feasible to distribute prebuilt binaries in future releases (#28).

Fixes #5.

bradlarsen and others added 30 commits March 17, 2023 16:20
GitHub Actions gives this:

    ERROR: docker exporter does not currently support exporting manifest lists

Probably the `docker push` logic needs to be changed so it's not a
separate step.

This needs additional investigation.
@bradlarsen bradlarsen marked this pull request as ready for review March 31, 2023 15:48
@bradlarsen bradlarsen merged commit 1dc4ce9 into main Mar 31, 2023
@bradlarsen bradlarsen deleted the vendor-vectorscan branch March 31, 2023 15:53
@bradlarsen bradlarsen added enhancement New feature or request packaging Related to packaging and distribution labels Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request packaging Related to packaging and distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support non-x86_64 by switching from Hyperscan to Vectorscan
1 participant