-
Notifications
You must be signed in to change notification settings - Fork 120
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
remove standalone visual studio solutions
- Loading branch information
Showing
14 changed files
with
0 additions
and
3,449 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
1176da2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't use cmake, how can I compile and debug this - or is it to be going "binary only" then?
1176da2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The recent changes from #82 don't have proper updates for handmade VS projects.
I just run buildVS2019.bat and then open VS solution generated by CMake.
If you require those handmade projects let's ask @yjh-styx to update them as well.
1176da2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Either that or if you use a recent VS version you can open the CMake project directly: https://docs.microsoft.com/en-us/cpp/build/cmake-projects-in-visual-studio?view=msvc-170
1176da2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Opening it directly only works if you have:
With the existing solutions, which actually got a fix by @yjh-styx in his commit, you can just have the build tools installed and run msbuild against the solution files - neither an IDE nor cmake is necessary, vs2017 works (I think 2015 did work, too, but that got deleted before - which is not that a big issue as you can manually downgrade the 2017 one to 2015.