Skip to content
This repository has been archived by the owner on Dec 26, 2021. It is now read-only.

Added Full CMake support + Fixed Errors #4504

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

CXCubeHD
Copy link

@CXCubeHD CXCubeHD commented Sep 8, 2021

No description provided.

@CXCubeHD
Copy link
Author

CXCubeHD commented Sep 8, 2021

Tested with CMake and VS

@richardletshacks
Copy link
Collaborator

the check fails though, might have to do with the submodule

@CXCubeHD
Copy link
Author

CXCubeHD commented Sep 8, 2021

@richardletshacks that's because of MinHook linking, for my mashine it worked without any problems. I will try to fix it soon or later.

@CXCubeHD
Copy link
Author

CXCubeHD commented Sep 8, 2021

@richardletshacks now it compiles without problems!

@richardletshacks
Copy link
Collaborator

please open separate pull requests, 163a17a shouldn't be included in this one.

@CXCubeHD
Copy link
Author

CXCubeHD commented Sep 8, 2021

@richardletshacks sry didn't read, but this is a minimal addition (that I need), so it shouldn't affect anything

@richardletshacks richardletshacks requested review from GodGamer029 and removed request for GodGamer029 September 10, 2021 15:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants