[LTP] [PATCH 0/2] lib: Add TINFO_WARN
Cyril Hrubis
chrubis@suse.cz
Wed May 29 17:10:35 CEST 2024
Hi!
> I'm not sure about this. Why not enabling TINFO + TWARN combination instead?
>
> tst_res(TINFO | TWARN, "my message");
That's even more confusing. Does that propagate into results or not?
To be honest we even have problem deciding if we should use TINFO or
TWARN in some cases and adding third variant would make things even
worse, sicne we would have three options.
So I would really keep just TINFO which is something that is printed by
default but does not propagate into results and TWARN that is printed as
well but propagates into results.
Maybe it would be even better to actually remove TWARN. That way we
would have only TFAIL and TBROK that propagate into results since TWARN
is kind of lesser TBROK anyways...
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list