geodynamics / aspect

A parallel, extensible finite element code to simulate convection in both 2D and 3D models.
https://aspect.geodynamics.org/
Other
223 stars 235 forks source link

Use TARGET_INCLUDE_DIRECTORIES instead of INCLUDE_DIRECTORIES. #5786

Open bangerth opened 4 months ago

bangerth commented 4 months ago

5777 introduces an include_directories call that just adds a directory to the include path of everything. That ought to be changed to a target_include_directories for only those targets that need that. I'll get to that after #5777 is merged, along with the comments in https://github.com/geodynamics/aspect/pull/5777#pullrequestreview-2097831299.