Put engine/ before other paths when including#75
Put engine/ before other paths when including#75WPettersson wants to merge 2 commits intoservice-7.0from
Conversation
Hopefully this will fix the problem where we sometimes pull in external libnormaliz headers.
|
@baburton, this fix lies in the build process for Regina's Python module, but it'll also potentially arise for any build process where:
Under this scenario, the compiler needs to be told to check the Regina header location before the "external" locations. This might apply to e.g. the GUI (which Nathan isn't building) and possibly the iOS apps. I don't know whether we want to cover those scenarios. It seems the engine is not susceptible, though. I'm guess that this is because if However, the root of the problem is slightly different. While digging, I noticed that We don't ship any libnormaliz headers, which technically means I think the shipped headers have been unusable for a while (possibly 6 years?). If we remove that include from |
Fix the problem where we sometimes pull in external libnormaliz headers.