Hi Matthieu, On Mon, Oct 19, 2020 at 5:47 PM Matthieu Baerts <matthieu.bae...@tessares.net> wrote: > On 19/10/2020 13:32, Geert Uytterhoeven wrote: > > MPTCP_KUNIT_TESTS selects MPTCP, thus enabling an optional feature the > > user may not want to enable. Fix this by making the test depend on > > MPTCP instead. > > I think the initial intension was to select MPTCP to have an easy way to > enable all KUnit tests. We imitated what was and is still done in > fs/ext4/Kconfig. > > But it probably makes sense to depend on MPTCP instead of selecting it. > So that's fine for me. But then please also send a patch to ext4 > maintainer to do the same there.
Thanks, good point. I didn't notice, as I did have ext4 enabled anyway. Will send a patch for ext4. Looks like ext4 and MPTCP where the only test modules selecting their dependencies. > Reviewed-by: Matthieu Baerts <matthieu.bae...@tessares.net> Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org 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