Skip to content

Move fully to CMake#781

Draft
rtm516 wants to merge 3 commits intosmartcmd:mainfrom
rtm516:feature/cmake-full
Draft

Move fully to CMake#781
rtm516 wants to merge 3 commits intosmartcmd:mainfrom
rtm516:feature/cmake-full

Conversation

@rtm516
Copy link
Collaborator

@rtm516 rtm516 commented Mar 7, 2026

Description

  • Removes all msbuild/Visual Studio requirements
  • Full CMake now
  • Also restructures files to match the old structure in the visual studio filters
  • Update actions script

This should also pave the way for builds on other platforms like linux

Changes

Previous Behavior

N/A

Root Cause

N/A

New Behavior

N/A

Fix Implementation

N/A

AI Use Disclosure

AI was used to generate the sources groups file from the filters as I really didnt wanna do that by hand.

@rtm516 rtm516 changed the title Feature/cmake full Move fully to CMake Mar 7, 2026
@Fivebuss
Copy link

Fivebuss commented Mar 7, 2026

How about make visual studio use cmake instead?

Keeping VS support while making cmake (righfully) the build system

@lightyisreal
Copy link

Would this allow the project to be built using MINGW? (Unless the project relies on something that only VS has)

@rtm516
Copy link
Collaborator Author

rtm516 commented Mar 7, 2026

How about make visual studio use cmake instead?

Keeping VS support while making cmake (righfully) the build system

It does, you just open the project folder with visual studio, I've updated the contributing guide to reflect this.

@rtm516
Copy link
Collaborator Author

rtm516 commented Mar 7, 2026

Would this allow the project to be built using MINGW? (Unless the project relies on something that only VS has)

Should do although untested

@bootsareme
Copy link
Contributor

I assume this still allows MSBuild to be used?

@rtm516
Copy link
Collaborator Author

rtm516 commented Mar 7, 2026

I assume this still allows MSBuild to be used?

I believe it will run the cmake scripts, either way it works fine on my windows box through VS so hopefully should on everyone elses

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.

4 participants