[LTP] [PATCH 2/2] tst_test.sh: Add public tst_test_root command
Cyril Hrubis
chrubis@suse.cz
Wed Oct 9 14:28:02 CEST 2019
Hi!
> >> 2. I wanted to make it consistent with other functions, that do similar
> >> stuff like
> >> ???? tst_test_drivers and tst_test_cmds. Both also call tst_brk in case
> >> of unsuccessful tests.
> > Well I do not like these names either, it's less descriptive that it
> > would have been with tst_require_cmds and tst_require_drivers, so if
> > anything I would be for renaming the tst_test_* ones.
> Fair enough, that makes sense. But then we require a change to
> tst_test_* first,
> otherwise the api names are inconsistent. And I'm not sure if changing the
> other functions is really worth it.
Should be easy for tst_test_drivers, that one is only used to implement
the NEEDS_DRIVERS variable. And for the second one, we would have to
change a few tests and one line of documentation, but that would be just
a simple script. I can do that if we agree on the change.
@Jan @Peter what do you think?
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list