BUILDING.md#44
Open
000MDK wants to merge 31 commits into
Open
Conversation
nukeykt chose sc55emu.exe as name since multiple SC-55 variants are now supported
adjusted formating, added dependencies to be on the safe side, added note on toolset selection
OpenRift412/000MDK app.ico for CMAKE_AUTORCC use
OpenRift412/000MDK app.ico
set(CMAKE_AUTORCC ON)
added note for building with VS2022
added infos from @eivindbohler ( nukeykt#28 (comment) )
|
|
Author
done, build guide updated |
Author
|
just tested the msvc-built .exe with roms too and looks like the window is just black and app stops responding, might just be a case of not enough SDL - investigating and updating |
|
I think you're getting confused. vcpkg is for downloading dependencies. If you download SDL2 separately and compile it, vcpkg is useless. You don't have to download SDL2 separately. I compiled a 64 bit build with Visual Studio a day ago, it works #28 (comment) |
built debug, now builds release
Author
|
yeah found that on my own in the mean time regarding vcpkg - changing example right now the change in CMakeList.txt is so you don't need to add anything anymore to the project if you did not notice btw |
getting SDL2 with vcpkg, needs more testing still
MSVS2022 instructions complete now with vcpkg, sdl2 and pkg-config MSYS2 now with less prereq
easier install
if pkgconf doesn't do it's automagic, the note will help setting SDL2_DIR manually
last changes before adding the scripts to it
cd .. not needed there
|
i think its best to not rely on scripts (sh) |
|
|
|
managed to fix my issues, now it compiles perfectly |
|
|
Author
looking into it, thanks for the test and the feedback |
|
I think the guide for MSVC is a little too complex... It's easier than that. ###First time only
###End 4a. Open 4b. Run everything from Visual Studio GUI. |
Author
|
this is a great addition 'cmake --toolchain C:/vcpkg/scripts/buildsystems/vcpkg.cmake -A x64 -B build` that I was overlooking today I'll install some VMs for having clean build/test systems (that's what the scripts are really for, they will go away eventually) and improve things from there in the next days |
|
added linux PR on your fork. output: |
|
one thing i can note is nuked-sc55 requires |
|
This worked for me on Linux Manjaro, with cmake and ninja 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Will add "Ready to title" after adding linux and some more testing on clean systems
added BUILDING.md as per #28
done:
todo: