Re: [PATCH] lib/tests: Make RANDSTRUCT_KUNIT_TEST depend on RANDSTRUCT

From: Geert Uytterhoeven
Date: Mon Jun 02 2025 - 14:14:52 EST


Hi Kees,

On Mon, 2 Jun 2025 at 20:08, Kees Cook <kees@xxxxxxxxxx> wrote:
> On Mon, Jun 02, 2025 at 04:49:51PM +0200, Geert Uytterhoeven wrote:
> > When CONFIG_RANDSTRUCT is not enabled, all randstruct tests are skipped.
> > Move this logic from run-time to config-time, to avoid people building
> > and running tests that do not do anything.
>
> I don't like doing this because it means that looking at CI output means
> I can't tell if the test was not built or if the config was not
> included. I want to always have the test available, but skip the test if
> the config is missing.

So should we drop all dependencies from tests?
Do you want Zorro bus, NuBus, ... tests (assumed we have them) to
be built on all platforms, and "run" on all CIs?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds