#768: Change per thorn -DTHORN_IS_xxx to a per thorn -I bindings/include/xxx --------------------------+------------------------------------------------- Reporter: sbrandt | Owner: Type: enhancement | Status: new Priority: optional | Milestone: Cactus_4.1.0 Component: Cactus | Version: Cactus_4.0.0 Resolution: | Keywords: build system --------------------------+-------------------------------------------------
Comment (by anonymous):
Thanks for the feedback.
This patch is actually still under development. While it does compile (not in Debug mode) it turns out Mojave was still confused by header files in common areas that included thorn specific files. That case turns out to have essentially the same problem as per thorn defines. You put your curser on "definethisthorn.h" and the browser doesn't know where to take you.
The newest version of the patch fixes this by (for example) making cctk.h a very small per-thorn include file that includes definethisthorn.h and a new header called cctk_core.h. I plan to attach the newer patch to this ticket when it's been tested a bit more.