Skip to content
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

Switch build system to CMake #39

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

LaserEyess
Copy link

@LaserEyess LaserEyess commented May 29, 2023

Description

Switches VS solution and makefile to cmake. I didn't change the switch port, and I left tcc as is, though you should be able to use that with cmake.

Will this Pull Request break anything?

Might break Windows, since I haven't tested that yet

Suggested Testing Steps

cmake -B build && cmake --build build --parallel and verify it still works

closes #13

@LaserEyess
Copy link
Author

@KiritoDv I'll probably need help testing the switch stuff

@LaserEyess LaserEyess force-pushed the cmake branch 3 times, most recently from 03de46f to e480022 Compare June 27, 2023 21:35
@LaserEyess LaserEyess marked this pull request as draft June 29, 2023 01:38
@LaserEyess
Copy link
Author

LaserEyess commented Jun 29, 2023

Broke windows builds somehow, I'll fix it this weekend.

  • MSVC 2022
  • mingw64
  • Linux
  • Switch

All working, and documentation is likewise updated for these changes.

@LaserEyess LaserEyess marked this pull request as ready for review July 3, 2023 01:32
@LaserEyess LaserEyess force-pushed the cmake branch 4 times, most recently from bf97a38 to 6d96c49 Compare July 3, 2023 16:21
This unifies the build system across platforms, including doing
development inside Visual Studio. No change to other code.

Some exceptions:
 - Switch port is still a Makefile
 - run_with_tcc.bat still exists
This unifies the build system across platforms, including doing
development inside Visual Studio. No change to other code.
@shinra-electric
Copy link

For what it's worth, this works on Mac too..

SM Terminal Output.txt

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.

Better build system?
2 participants