Skip to content

Build improvements#53

Merged
crud89 merged 8 commits intomainfrom
build-improvements
Jul 8, 2021
Merged

Build improvements#53
crud89 merged 8 commits intomainfrom
build-improvements

Conversation

@crud89
Copy link
Copy Markdown
Owner

@crud89 crud89 commented Jul 8, 2021

This PR changes the build script slightly in a way, that vcpkg can more easily handle building the engine. When using the registry, currently the portfile needs to apply a patch before the build completes successfully. With this PR, the patch should be no longer needed.

In particular this PR:

  • Moves the toolchain file definition out of the top-level CMakeLists.txt (into the CMakeSettings.json file).
  • Removes files that configure into the source directory (i.e. headers of the core project).
  • Fixes an issue, that caused glm and DirectX Math to still be linked, even if the dependencies should be disabled.
  • Initializes install directories from CMake defaults.

@crud89 crud89 added Priority: Medium A issue with normal priority. Type: Requirement Build 🛠 Issues that involve the build process. labels Jul 8, 2021
@crud89 crud89 added this to the Alpha #02 milestone Jul 8, 2021
@crud89 crud89 self-assigned this Jul 8, 2021
@crud89 crud89 merged commit 7e8dedb into main Jul 8, 2021
@crud89 crud89 deleted the build-improvements branch July 8, 2021 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build 🛠 Issues that involve the build process. Priority: Medium A issue with normal priority.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant