Hmm, not a fan of removing the options entirely if ZYDIS_ROOT_PROJECT isn't set. This will probably break vcpkgs build which itself uses CMake to describe package builds (so Zydis won't be top-level project).

I'd simply change the default values of those properties that are currently ON to ${ZYDIS_ROOT_PROJECT} instead. This sidesteps the issue and should provide the same desired outcome.

I suggested doing it this way to as well reduce option pollution while we are already on it.

Not sure about vcpkg, but that needs to be checked first. I'm fine with your approach as well, if removing the options might break some stuff.

Cmake itself is pretty forgiving with the options. Not declared ones are simply assumed to be OFF.

I've defaulted some of the options that were ON by default to ${ZYDIS_ROOT_PROJECT}, this gives us the same result but keeps the options alive and one can still override them with cmake command line using -D, so I think this is the most optimal approach.

I suggested doing it this way to as well reduce option pollution while we are already on it.

I don't think that our options are excessive. I far prefer having all the options visible over the user having to figure out why the hell some options magically disappeared once they include Zydis into their project.

Fine for me!

Maybe let's do the same change in Zycore and update the submodule as part of this PR before merging?

Zycore doesn't appear to have any default-ON options, so there shouldn't be any change required there. Or would you like to default-enable the tests there with a similar switch? That'd be fine with me.

Yeah it's inconsistent at the moment. Would be better to have the same logic in Zycore as well. But as the tests are currently OFF for Zycore, we can as well merge right away 🙂