Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vcpkg support for VS, delete unneeded files #796

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

svew
Copy link

@svew svew commented Oct 4, 2023

vcpkg

  • Added vcpkg support for VS. All you should need now is to install vcpkg on your compute, configure the project, and the project will automatically download, install, and link packages
  • Deleted src/imgui/, as it was a library that could be pulled from vcpkg
  • Updated header references to match vcpkg folder structure

Miscellaneous

  • Aggressively cleaned up the Barony.vcxproj file to make it more readable. Project configurations were extremely hard to diff before.
    • Assembly references, especially for fmod, may not work the same
    • Removed the after build scripts that copied binaries in unpredictable ways
  • Added MSBuild target that will automatically copy over resource files from your game installation. You need to set the BARONY_GAME_DIR environment variable for it to work.
  • Replaced rapidjson "GetObject()" calls with "GetObj()", because "GetObject()" is also a windows macro that was breaking the build

Todo

  • Verify sure that fmod references work correctly
  • Verify that all build configurations build and are accurate

Future Work

  • Configure editor.vcxproj similarly
  • Make CMake use vcxproj as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant