-
Notifications
You must be signed in to change notification settings - Fork 9
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
Specify Zeek scripts to install explicitly. #77
Comments
bbannier
added a commit
that referenced
this issue
Sep 9, 2021
This patch adds a new CMake function `spicy_analyzer` takes similar arguments as `spicy_add_analyzer`, but which can additionally accept a list of scripts to install via a `SCRIPTS` parameter. In order to support different directories for file and protocol analyzer scripts this function also requires explicitly passing in an analyzer `KIND` (for our current use cases: either `file` or `protocol`). Closes #77.
bbannier
added a commit
that referenced
this issue
Sep 9, 2021
This patch adds a new CMake function `spicy_analyzer` takes similar arguments as `spicy_add_analyzer`, but which can additionally accept a list of scripts to install via a `SCRIPTS` parameter. In order to support different directories for file and protocol analyzer scripts this function also requires explicitly passing in an analyzer `KIND` (for our current use cases: either `file` or `protocol`). Closes #77.
bbannier
added a commit
that referenced
this issue
Sep 10, 2021
With this patch we now use named arguments to `spicy_add_plugin` which allows us to pass additional Zeek scripts to install. Closes #77. Don't install. This reverts commit 59a65a185b81e2ece29aeb62462201a42fdf0c55.
bbannier
added a commit
that referenced
this issue
Sep 10, 2021
With this patch we now use named arguments to `spicy_add_plugin` which allows us to pass additional Zeek scripts to install. Closes #77. Don't install. This reverts commit 59a65a185b81e2ece29aeb62462201a42fdf0c55.
bbannier
added a commit
that referenced
this issue
Sep 13, 2021
With this patch we now use named arguments to `spicy_add_plugin` which allows us to pass additional Zeek scripts to install. Closes #77.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now we do a blanket install of all
*.{zeek,sig}
. Fromanalyzer/CMakeLists.txt
:The text was updated successfully, but these errors were encountered: