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

Fix usage vcpkg for windows and add cmake preset for vscode #414

Conversation

Ygrik2003
Copy link
Contributor

@Ygrik2003 Ygrik2003 commented Dec 14, 2024

Добавил пресет для интеграции с VSCode и подправил cmake для правильной работы vcpkg

Теперь не нужно 2 раза вызывать конфигурацию. Избавился от параметра VOXELENGINE_BUILD_WINDOWS_VCPKG в CMake. Теперь по дефолту windows билд собирается с vcpkg

Избавился от vcpkg загрузки из CMake, теперь есть ограничения:

  1. Должна быть выставлена переменная среды VCPKG_ROOT, указывающая на корень установки vcpkg
  2. Собственно сам vcpkg нужно ставить самому как в доке

За одно подправил деплой ресов, теперь он всегда будет работать под все платформы без проблем и костылей

Единственное, не уверен, что не сломал сборку windows-clang.yml, нужно перепроверить, возможно нужно делать экспорт полного пути к vcpkg

.github\workflows\windows-clang.yml:46

export VCPKG_ROOT=./vcpkg
export VCPKG_DEFAULT_TRIPLET=x64-mingw-static
export VCPKG_DEFAULT_HOST_TRIPLET=x64-mingw-static
mkdir build
cd build
cmake --preset default-windows -G "MinGW Makefiles" -DVCPKG_TARGET_TRIPLET=x64-mingw-static -DCMAKE_BUILD_TYPE=Release ..
cmake --build . --config Release

В любом случае нужно перепроверить этот билд

@Ygrik2003
Copy link
Contributor Author

К слову, потыкался чутка с vcpkg, оказывается, если запускаться в дев терминале вижлы, то там прокидывается переменная VCPKG_ROOT автоматически, так что можно еще будет дописать в readme, вечером закомитаю

@MihailRis
Copy link
Owner

Обновил настройки workflow в ветке, чтобы запускались проверки в PR. Нужно вытянуть изменения из headless-mode.

Ygrik2003 and others added 3 commits December 20, 2024 15:39
…code' of github.com:Ygrik2003/VoxelEngine-Cpp into Fix-usage-vcpkg-for-windows-and-add-cmake-preset-for-vscode
@Ygrik2003 Ygrik2003 marked this pull request as draft December 20, 2024 14:11
@Ygrik2003
Copy link
Contributor Author

Еще немного потрогал Github Actions, теперь билды будут работать прямо из пресетов, что должно чутка облегчить работу с workflows при работе с флагами
Сделал clang сборку через генератор Ninja Multi-Config
Переделал подключение vcpkg в github actions в соответствии с докой
в общем нужно перепроверить билды

@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 20, 2024 14:23
@Ygrik2003
Copy link
Contributor Author

image

@Ygrik2003
Copy link
Contributor Author

Ygrik2003 commented Dec 20, 2024

Отвалились ctest и неправильные пути для msys2 под clang, завтра-послезавтра гляну

@Ygrik2003 Ygrik2003 marked this pull request as draft December 20, 2024 22:11
@Ygrik2003
Copy link
Contributor Author

Короче можно попробовать первоначальный фикс с VCPKG_ROOT: ./vcpkg
если не сработает, то нужно будет думать

@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 20, 2024 22:35
@Ygrik2003 Ygrik2003 marked this pull request as draft December 21, 2024 07:22
@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 25, 2024 00:18
@MihailRis
Copy link
Owner

Добавил тест производительности.
Нужно проверить перед слиянием.

@Ygrik2003
Copy link
Contributor Author

Подмержил

@Ygrik2003 Ygrik2003 marked this pull request as draft December 25, 2024 13:47
@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 25, 2024 16:11
@Ygrik2003
Copy link
Contributor Author

Откатился до msys2, перфа восстановилась, видимо нужно играть с флагами llvm clang'а (хотя теперь я и вовсе не уверен, что можно достичь такой перфы, зная, что llvm clang все равно юзает MSVC), а для этого хотелось бы профилировщик заиметь, что бы хотя бы понимать, в каком направлении можно двигаться

@Ygrik2003 Ygrik2003 marked this pull request as draft December 25, 2024 16:18
@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 25, 2024 16:22
@Ygrik2003 Ygrik2003 marked this pull request as draft December 25, 2024 16:32
@Ygrik2003 Ygrik2003 marked this pull request as ready for review December 25, 2024 16:35
@Ygrik2003 Ygrik2003 requested a review from MihailRis December 25, 2024 22:06
@MihailRis MihailRis merged commit 6cb0d31 into MihailRis:headless-mode Dec 26, 2024
4 checks passed
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.

2 participants