Skip to content

Add CMake options for more build granularity#233

Merged
markos merged 1 commit intoVectorCamp:developfrom
bradlarsen:develop
Mar 12, 2024
Merged

Add CMake options for more build granularity#233
markos merged 1 commit intoVectorCamp:developfrom
bradlarsen:develop

Conversation

@bradlarsen
Copy link
Copy Markdown

I vendor Vectorscan as part of Nosey Parker. In that use case, I only build vectorscan for its runtime system, and don't build the tests, docs, standalone tools, or a few other pieces. I've had to patch Vectorscan locally to exclude several parts from the build.

Some parts that I have disabled may be useful for other users of Vectorscan, so I've put them in a pull request here.

This pull request adds three new CMake options, all defaulting to true, making it possible to opt-out of building parts of Vectorscan that are not essential for deployment of the matching runtime.

These new options:

  • BUILD_UNIT: control whether the unit directory is included
  • BUILD_DOC: control whether the doc directory is included
  • BUILD_TOOLS: control whether the tools directory is included

This adds three new CMake options, all defaulting to true, making it
possible to opt-out of building parts of Vectorscan that are not
essential for deployment of the matching runtime.

These new options:

- `BUILD_UNIT`: control whether the `unit` directory is included
- `BUILD_DOC`: control whether the `doc` directory is included
- `BUILD_TOOLS`: control whether the `tools` directory is included
Copy link
Copy Markdown

@markos markos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, approved.

@markos markos merged commit 9db7b52 into VectorCamp:develop Mar 12, 2024
markos added a commit that referenced this pull request Oct 24, 2025
Add CMake options for more build granularity
markos added a commit that referenced this pull request Oct 24, 2025
Add CMake options for more build granularity
markos added a commit that referenced this pull request Oct 24, 2025
Add CMake options for more build granularity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants