X Tutup
Skip to content

Option to pass extra arguments to ctest#315

Open
kalebealvs wants to merge 3 commits intoCivitasv:masterfrom
kalebealvs:ctest_args
Open

Option to pass extra arguments to ctest#315
kalebealvs wants to merge 3 commits intoCivitasv:masterfrom
kalebealvs:ctest_args

Conversation

@kalebealvs
Copy link

@kalebealvs kalebealvs commented May 2, 2025

This patch enables the plugin to pass extra args to ctest when running tests.

Resolves: #296

Civitasv
Civitasv reviewed May 14, 2025
View reviewed changes
README.md
require("cmake-tools").setup {
cmake_command = "cmake", -- this is used to specify cmake command path
ctest_command = "ctest", -- this is used to specify ctest command path
ctest_extra_args = {}, -- this is used to specify extra args for ctest
Copy link
Owner

@Civitasv Civitasv May 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can put this variable into Config.base_settings? So we can edit it easily.

Copy link
Author

@kalebealvs kalebealvs Oct 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea. Done.

@Civitasv
Copy link
Owner

Civitasv commented May 14, 2025

Sorry for my late. It's a great option, I personally think putting it into cmake_settings will be better. What do you think?

@kalebealvs
Copy link
Author

kalebealvs commented Oct 4, 2025

Sorry for my late. It's a great option, I personally think putting it into cmake_settings will be better. What do you think?

I'm so sorry. I completely forgot about this since I was using my fork and didn't pay much attention to the notifications.
Do you mean creating a new object called cmake_settings and putting ctest_extra_args under it?
That being the case, isn't this root object kinda like "cmake settings" already, since there are other things here also tied to cmake settings, like regenerate_on_save, generate_options, build_directory, among others?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Civitasv Civitasv Civitasv left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Option to pass extra arguments to ctest

2 participants

@kalebealvs @Civitasv
X Tutup