Enable windows build and add docs about it #4
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds Windows platform support and improves asset path resolution for the ConsoleGame project. The changes enable the game to build and run on Windows using vcpkg for dependency management, while maintaining existing Linux/macOS support.
Key changes:
- Introduced cross-platform asset path resolution system that searches multiple candidate locations
- Added platform-specific font fallback mechanism for Windows and Linux/macOS
- Updated CMake configuration to support both Windows (vcpkg) and Linux/macOS (pkg-config) build systems
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/main.cpp | Updated main function signature to accept command-line arguments with proper attributes |
| src/Game.cpp | Added asset path resolution helpers and improved font loading with multiple fallback candidates |
| README.md | Added comprehensive Windows build instructions, vcpkg setup guide, and troubleshooting section |
| CMakeLists.txt | Implemented platform-specific dependency finding and compiler option configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - Or copy DLLs to the same directory as the executable | ||
|
|
||
| **Visual Studio build fails with "missing dependencies"** | ||
| - Reinstall vcpkg libraries: `vcpkg remove sdl2:x64-windows sdl2-image:x64-windows box2d:x64-windows` |
There was a problem hiding this comment.
The reinstall command is missing 'sdl2-ttf:x64-windows' in the removal list, which is inconsistent with the installation command on line 151 that includes it. Add sdl2-ttf to this removal command for consistency.
| - Reinstall vcpkg libraries: `vcpkg remove sdl2:x64-windows sdl2-image:x64-windows box2d:x64-windows` | |
| - Reinstall vcpkg libraries: `vcpkg remove sdl2:x64-windows sdl2-image:x64-windows sdl2-ttf:x64-windows box2d:x64-windows` |
michael-stanin
force-pushed
the
feature/windows-build
branch
from
c194dc7 to
55c88b5
Compare
No description provided.