TiManGames / AmnesiaTheDarkDescent

GNU General Public License v3.0
30 stars 8 forks source link

CMake: all dependency headers are included from OALWrapper #46

Closed jkulawik closed 6 days ago

jkulawik commented 3 weeks ago

The entire include folder is added in the OALWrapper CMake instead of including sub-folders as needed.

This is a "best practices" thing so not very important, but the appropriate dirs should probably be included in the same places as the libs are. Doing this has a potential to reduce compilation issues.

jkulawik commented 3 weeks ago

Actually this is done twice - once in OALWrapper and once in the core CMake. So while worth cleaning up, it's something the game did on purpose anyway.