On 02/12/2019 23:32, Chris Johns wrote:
On 2/12/19 6:24 pm, Sebastian Huber wrote:Use the value expected by the RTEMS Tester. There are two separator characters used ('-' and '_') for the states. This is a bit inconsistent.Sorry about this. I had not noticed.--- cpukit/libtest/testbeginend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cpukit/libtest/testbeginend.c b/cpukit/libtest/testbeginend.c index b739e47b2e..9bdc86780c 100644 --- a/cpukit/libtest/testbeginend.c +++ b/cpukit/libtest/testbeginend.c @@ -29,7 +29,7 @@ rtems_printer rtems_test_printer = { static const char* const test_state_strings[] = { "EXPECTED-PASS",Should this change or should `USER_INPUT` be changed and we update ... https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n159 https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n161 ? It would be a simpler change because this change means we move to having 2 with `_` and a bunch with `-`. There are internal states in the tester such as `WRONG-VERSION` and changing all of those is a bigger change in the tester.
If we update the tester, then it should accept '-' and '_' versions so that it can be used for test runs before and after a change of cpukit/libtest/testbeginend.c.
-- Sebastian Huber, embedded brains GmbH Address : Dornierstr. 4, D-82178 Puchheim, Germany Phone : +49 89 189 47 41-16 Fax : +49 89 189 47 41-09 E-Mail : [email protected] PGP : Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG. _______________________________________________ devel mailing list [email protected] http://lists.rtems.org/mailman/listinfo/devel
