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

CMake Improvements #14

Closed
mondus opened this issue Jun 1, 2023 · 3 comments · Fixed by #67
Closed

CMake Improvements #14

mondus opened this issue Jun 1, 2023 · 3 comments · Fixed by #67
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@mondus
Copy link
Contributor

mondus commented Jun 1, 2023

We should add some CMake flags to improve the build experience this should include

  • Examples: A flag for each to enable building
  • Documentation: We should remove the dependencies for documentation building if only building the library
  • Testing: We should remove the dependencies for testing if only building the library
@mondus mondus added this to the Initial Release milestone Jun 1, 2023
@jatkinson1000
Copy link
Member

@SimonClifford has pointed out that we do not quite match the 'ideal' cmake installation structure meaning that we have to 'overspecify' directories when linking etc. This can be fixed and the CMake flags and commands made simpler.

@jatkinson1000
Copy link
Member

We might also tie this in with #9 (or at least be aware of it) seeking to add CUDA as a language to CMake to make building with GPU an option.

@jatkinson1000
Copy link
Member

@ElliottKasoar is this something you might fancy taking a look at after #38 ?

@jatkinson1000 jatkinson1000 mentioned this issue Nov 19, 2023
@jatkinson1000 jatkinson1000 self-assigned this Nov 19, 2023
@jatkinson1000 jatkinson1000 added bug Something isn't working enhancement New feature or request labels Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants