caseymcc / UE4CMake

Provides a simple way to add a cmake lib to any Unreal Engine 4 (UE4) or 5 (UE5) project.
MIT License
78 stars 14 forks source link

Take interface include directories into account for targets #5

Closed fsvenson closed 1 year ago

fsvenson commented 1 year ago

It looks like currently include directories are only pulled from the INCLUDE_DIRECTORIES property of a target. One would expect that INTERFACE_INCLUDE_DIRECTORIES and possibly INTERFACE_SYSTEM_INCLUDE_DIRECTORIES should be added to the include path as well.

caseymcc commented 1 year ago

Added INTERFACE_INCLUDE_DIRECTORIES to includes, Not sure about the INTERFACE_SYSTEM_INCLUDE_DIRECTORIES, but if someone comes in with a case that needs it it can be added