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

Use cmake #33

Closed
wants to merge 10 commits into from
Closed

Use cmake #33

wants to merge 10 commits into from

Conversation

HenryHeffan
Copy link

This fixes the naming conventions of headers and import paths, and makes the build system much easier to understand and change. It now uses cmake. To the first time you want to build the program, do the following for the root directory in the project

$ mkdir build_dir
$ cd build_dir
$ cmake ..
$ cmake --build .

for subsequent builds, from build_dir run cmake --build . -- -j 8 (or a different number depending on number of cores desired)
To install the files, from build_dir run cmake --install . --prefix /path/to/install/directory

I know you might not accept this pull request but at least fetch the branch and try it before rejecting it. I think it will make development simpler.

@HenryHeffan
Copy link
Author

Im not sure how to get the automated builder to use the new build system to test it

Remove cluttering files
@HenryHeffan HenryHeffan force-pushed the use-cmake branch 16 times, most recently from b2bc025 to a097a05 Compare November 29, 2021 22:50
@HenryHeffan HenryHeffan force-pushed the use-cmake branch 6 times, most recently from 79d5b1c to 299305f Compare November 30, 2021 00:44
@HenryHeffan
Copy link
Author

I got it to build!

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.

1 participant