CMake: add all headers to target sources - Githubissues

DescentDevelopers / Descent3

Descent 3 by Outrage Entertainment

GNU General Public License v3.0

2.88k stars 251 forks source link

CMake: add all headers to target sources #517

Closed Lgt2x closed 3 months ago

Lgt2x commented 3 months ago

This can help with CMake IDE integration and file indexing.

Pull Request Type

[ ] GitHub Workflow changes
[ ] Documentation or Wiki changes
[x] Build and Dependency changes
[ ] Runtime changes
- [ ] Render changes
- [ ] Audio changes
- [ ] Input changes
- [ ] Network changes
- [ ] Other changes

Description

Set every missing header as source of CMake targets. This can improve IDE integration, and is generally a good practice to follow.

Related Issues

Mentioned in #502 https://github.com/DescentDevelopers/Descent3/pull/502#discussion_r1697584916

This should probably cover what's left to do out of #411

winterheart commented 3 months ago

No runtime issues found.