-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: MinGW compatible build. #562
base: master
Are you sure you want to change the base?
Conversation
chore(logger.cpp): Remove <format> include. refactor(CMakeLists.txt): Tabs to spaces.
working-directory: build/Release/
feat(CMakeLists.txt): Remove debug build.
chore(README.md): Fix typo.
fix(g3log): Ignoring options.
chore(CMakeLists.txt): Remove "Disable warnings" (Needs test in CI). refactor(CMakeLists.txt): Change "BigBaseV2" to "YimMenu". chore: Remove g_is_steam check.
fix(CI): Fix CI trying to access `BigBaseV2.dll` instead of `YimMenu.dll`.
…to feat-cmake-build. That was easier then i expected.
[BigBaseV2](https://github.com/Pocakking/BigBaseV2) mention with [UC](https://www.unknowncheats.me/forum/grand-theft-auto-v/476972-yimmenu-1-63-a.html) one. I hope markdown works.
…to feat-cmake-build.
…/TupoyeMenu into feat-mingw-build.
fix(MSVC): Added some `#if _MSVC_VER` checks.
…to feat-mingw-build. The build will fail, just committing in case i will try to update it again.
chore!: Removed pointers.
…to feat-mingw-build.
I had to remove signature scanning from Is there any other way to implement ptr cache? |
chore: Fix some warnings. Untested.
chore(CI): A little more uniform CI. Untested.
Download the artifacts for this pull request: |
Does not link, but at least it compiles fine... I hope. This will probably fail on GitHub because it does not have clang installed. Untested.
…to feat-mingw-build.
…to feat-mingw-build.
…to feat-mingw-build. Untested.
Probably works, howewer the GUI isn't rendering. Only the mouse cursor shows.
…to feat-mingw-build.
What's the reason to not use msvc-x64-cmake provided by the msvc-wine-git aur package and similar packages? I've had multiple people with Linux under arch systems and similar successfully compiling |
I already use msvc-wine, but i want clangd language server and i don't want to run wine all the time. Right now I'm using clang with msvc headers to compile YimMenu, and the main problem was Ah and also i think cpr did not work with msvc-wine some time ago, but maybe i did something wrong. |
I see, you can fix up pointers.cpp by explicitly using a compile time string struct ( there is one in common.hpp) and not using inline struct declaration inside the template parameters or at least be explicit about the constructor should fix it. For Lua imgui bindings they work under msvc wine, unsure about the other compilers |
fix(MinGW): Attent to fix MinGW not building. MinGW fix is untested.
fix(MinGW): Fixed some mingw error, not that it will build lua without it. Untested.
Reason
MSVC is windows only and using a VM to compile this menu every time you change something isn't nice.
Related:
Additional context
This is what i do to compile it (copy-pasted from CI)
P.s: Some commits from my cmake PR are visible here sorry about that.