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

Don't enable C in the tests #111

Open
zachcran opened this issue Sep 4, 2024 · 0 comments
Open

Don't enable C in the tests #111

zachcran opened this issue Sep 4, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@zachcran
Copy link
Contributor

zachcran commented Sep 4, 2024

Is your feature request related to a problem? Please describe.
No. I just saw this while working on #110 and figured I would suggest it.

Describe the solution you'd like
In the template file for the CMakeLists.txt of each test, the LANGUAGES of the project() command is set to C so "dummy libraries don't complain about not having a linkage language". I'm pretty sure this can be set to NONE to avoid enabling unnecessary languages in the tests.

Describe alternatives you've considered
Leave it as it is.

Additional context
I'm not sure what happens if there is not a C compiler on the system, nor do I know if fix would save any time. It just seems unnecessary since unit tests for CMake do not need C and you can set it to NONE without CMake complaining (as far as I could tell).

@zachcran zachcran added the enhancement New feature or request label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant