Skip to content

Update build process to use CMake instead of qmake - #16

Merged
jmw182 merged 15 commits into
mm_refactor_with_mapsfrom
cmake-build
Nov 9, 2026
Merged

jmw182 merged 15 commits into
mm_refactor_with_mapsfrom
cmake-build

Conversation

@jmw182

@jmw182 jmw182 commented Aug 31, 2026

Copy link
Copy Markdown
Member

Use cmake instead of qmake and sln files.

Update OS_defines.h and OS macros to not require USE_LINUX input to compiler.

Co-authored with copilot GPT-5.6 Terra.


Stack created with GitHub Stacks CLI • Give Feedback 💬

@jmw182

jmw182 commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

@dmweir I haven't tested this myself yet (will do so tomorrow), but I burned through some end-of-the-month tokens to finally replace the qmake dependency. I did this on linux (WSL), so copilot tested the build on linux. I will need to test on windows.

@jmw182

jmw182 commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

Copilot also added an actions workflow which I still need to review but it's failing for macos and passing on windows/linux.

@jmw182

jmw182 commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

If we decide to approve this branch I might try to move it down the stack. That would cause the other PR branches to rebase and include this update.

@jmw182

jmw182 commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

Tested build on Windows and WSL, CI passes on Windows, Linux, and MacOS. This much needed update looks good to me.

@dmweir
dmweir self-requested a review September 2, 2026 19:53
dmweir
dmweir previously approved these changes Sep 2, 2026
@jmw182
jmw182 removed this pull request from stack #13 November 9, 2026 15:36
@jmw182
jmw182 added this pull request to stack #18 November 9, 2026 15:37
@jmw182
jmw182 merged commit 8044d6f into v4 Nov 9, 2026
6 checks passed
@jmw182
jmw182 deleted the cmake-build branch November 9, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants