PacktPublishing / Mastering-Graphics-Programming-with-Vulkan

MIT License
514 stars 65 forks source link

failed to build Project1.sln #10

Closed JiangWZW closed 1 year ago

JiangWZW commented 1 year ago

Error when building project1.sln with Visual Studio 2019. Seems caused by the path "project..\binaries"

Other projects get built just fine, except for project 1.

_Severity Code Description Project File Line Suppression State Error MSB3073 The command "copy /Y F:\Mastering-Graphics-Programming-with-Vulkan\project..\binaries\SDL2-2.0.18\lib\x64\SDL2.dll F:\Mastering-Graphics-Programming-with-Vulkan\project..\binaries\ copy /Y F:\Mastering-Graphics-Programming-with-Vulkan\project..\binaries\assimp\windows\bin\assimp-vc142-mt.dll F:\Mastering-Graphics-Programming-with-Vulkan\project..\binaries\ :VCEnd" exited with code 1. RaptorEngine_Chapter1 C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Microsoft\VC\v160\Microsoft.CppCommon.targets 155

theWatchmen commented 1 year ago

Thanks for reporting this. Unfortunately I am not able to reproduce this issue. I am opening the solution project\RaptorEngine.sln and compiling RaptorEngine_Chapter1. I also double checked that the copy path is resolved correctly in the post build step: image Could you double check that this is what you're seeing as well? Which version of Visual Studio are you using?

Radagan commented 1 year ago

I ran into this problem on Debian LInux this morning. It looks like the pull includes only Windows binaries. If anyone else has this issue, it is a simple fix to add these from apt.

sudo apt install libsdl2-2.0-0
sudo apt install libsdl2-dev
sudo apt install libassimp5
sudo apt install libassimp-dev

@theWatchmen, maybe add this to the Linux section?

theWatchmen commented 1 year ago

Thanks lettings us know! We do provide the assimp library for Linux (https://github.com/PacktPublishing/Mastering-Graphics-Programming-with-Vulkan/tree/main/binaries/assimp/linux/lib) and CMake should reference that. I'll double check that we didn't forget to upload the SDL binaries for Linux.

Radagan commented 1 year ago

@theWatchmen, my bad. You do indeed! I was apt getting SDL, so I just got them both. ;^)

theWatchmen commented 1 year ago

No worries :) I added the instructions to install the SDL2 library to the README as that library is not provided with the repo.

theWatchmen commented 1 year ago

Closing this because of no recent activity.