Ben Elliston wrote:
On Fri, Jan 11, 2019 at 10:38:05PM -0600, Jacob Bachmeyer wrote:
Is the contents of the "runtests" variable part of the DejaGnu API
currently? The variable is not documented and runtest_file_p exists
to abstract its meaning.
No. It should only be procs. We should be providing accessor procs for
any variables, ideally.
The "testsuite file" command provides this for the *dir variables, but
the "testsuite" multiplex call is intended for commands relating to the
testsuite as a whole (another one on my TODO list is "testsuite depend"
or "testsuite depends" for tracking dependencies not directly visible to
the framework).
I have been trying to make a distinction between "testsuite options" and
"test run options". Essentially, a "testsuite option" is something
related to the code in a testsuite and simply setting it probably will
not work or be useful in general. The "MULTIPASS" variable is an
example of a testsuite option, as would be a future "API level"
declaration. A "test run option" is something that users can
more-or-less freely set, like --verbose or the name of a compiler binary
to use.
In this model, the "testsuite" command should probably stick to
"testsuite options", although "testsuite file" indirectly accesses the
*dir variables, which are "test run options", so the question then
becomes what to name the new API call. I will start with offering
"testsuite script args" defined to return what is currently the second
element in $runtests. If we also want the first element of $runtests
(and that is not entirely redundant to [info script] or we want it
anyway), I suggest "testsuite script name" for that purpose.
Comments?
-- Jacob
_______________________________________________
DejaGnu mailing list
DejaGnu@gnu.org
https://lists.gnu.org/mailman/listinfo/dejagnu