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

Support pointing BROKER_ROOT_DIR at Zeek build tree #108

Closed
wants to merge 1 commit into from

Conversation

J-Gras
Copy link

@J-Gras J-Gras commented May 10, 2024

I think this is needed if you want to point BROKER_ROOT_DIR into Zeek's build tree after zeek/broker@3c64a31.

@J-Gras J-Gras force-pushed the topic/jgras/broker-path-fix branch from a2ad940 to b30aa5d Compare May 10, 2024 09:24
@Neverlord
Copy link
Member

That script is obsolete. We guard against CMake picking this up by accident: https://github.com/zeek/zeek/blob/master/src/ZeekConfig.cmake.in#L30. Is that guard not working for you? How did you end up in that script?

@J-Gras
Copy link
Author

J-Gras commented May 10, 2024

Oh, I am using that script in an external project to find Broker. Is there another way to do that now?

@Neverlord
Copy link
Member

The "proper" CMake way is to use the CMake package config files. If there a legacy Find<PackageName> script on your system, you can pass CONFIG to find_package to tell CMake to not fall back on the legacy scripts ("config mode": https://cmake.org/cmake/help/latest/command/find_package.html).

That script definitely should no longer exists and I actually though we've removed it a while ago. Sorry for the inconvenience!

@J-Gras
Copy link
Author

J-Gras commented May 10, 2024

Thanks 🙏 I'll close this and reach out if I run into any issues implementing the proper integration.

@J-Gras J-Gras closed this May 10, 2024
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