Skip to content

meson: allow passing arguments to generate_test_runner.rb#826

Closed
lovetheguitar wants to merge 1 commit intoThrowTheSwitch:masterfrom
lovetheguitar:patch-1
Closed

meson: allow passing arguments to generate_test_runner.rb#826
lovetheguitar wants to merge 1 commit intoThrowTheSwitch:masterfrom
lovetheguitar:patch-1

Conversation

@lovetheguitar
Copy link
Copy Markdown

@lovetheguitar lovetheguitar commented Mar 26, 2026

This change enables a user to configure the test generation, e.g. enable parametrized tests like so:

gen_test_runner.process(
      'test_platform.c' , extra_args: ['--use_param_tests=1']

This feature is documented here in the meson documentation.

Btw. thanks for the great library. :)

This change enables a user to configure the test generation, e.g. enable parametrized tests like so:

```meson
gen_test_runner.process(
      'test_platform.c' , extra_args: ['--use_param_tests=1']
```
@lovetheguitar
Copy link
Copy Markdown
Author

Whoopsie, there is an existing open PR - please consider #777 instead.

Sorry for the noise. 🙃

@Letme
Copy link
Copy Markdown
Contributor

Letme commented Mar 27, 2026

@lovetheguitar can you review #777 and put approval to it if you agree with it? I tried meson, but never got it working, so it is safe to say that if more people approve that MR Mark will merge it sooner :) (it does not work like that but still nice to have some confidence)

@mvandervoord
Copy link
Copy Markdown
Member

@Letme @lovetheguitar -- I mean, it DOES sort-of work like that? For features that I don't use and don't have a simple way to validate, it's definitely nice to have others give a thumb-up that they like it! Bonus points if they express that they actually tried it and it works! I appreciate your help (and patience!)

@Letme
Copy link
Copy Markdown
Contributor

Letme commented Mar 27, 2026

@mvandervoord I know it does, but fact is there are many open Pull requests there, where you cannot test them and they probably work, but because there aren't any non-authors confirming they need it, they just get stuck and then soon forgotten. So I understand you need to be pinged, and it is much better if you get pinged only when there is approval and work mostly completed.

@lovetheguitar
Copy link
Copy Markdown
Author

@Letme Yes I did, it works fine. :)

@mvandervoord Great, happy to get&give feedback. Will definitely include some output trying to showcase the difference next time. 👍

I also verified #764 ✔️ :

Before

  Subprojects
    unity: YES 1 warnings

After

  Subprojects
    unity: YES

Short question regarding releases, is there any procedure or schedule how/when something gets released?
Thanks, I appreciate the responsiveness a lot!

@mvandervoord
Copy link
Copy Markdown
Member

We are attempting to get to a more regular release cadence. Ceedling has had the largest backlog of needs for the last couple years, and so much of our focus has been on major cleanup/improvements to that. We're just now getting back to a point where we can focus on the foundation projects again. In all cases, our dream is to make supporting these into our fulltime jobs... but at the moment they are sadly free time driven. ;)

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.

3 participants