On 04/12/2019 12:59, Joel Sherrill wrote:
On Wed, Dec 4, 2019, 3:57 AM Sebastian Huber <[email protected] <mailto:[email protected]>> wrote:On 04/12/2019 09:49, Chris Johns wrote: > On 4/12/19 5:43 pm, Sebastian Huber wrote: >> On 04/12/2019 05:14, Chris Johns wrote: >>> On 3/12/19 9:59 pm, Sebastian Huber wrote: >>>> 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. >>> Yes this is a good point. Shame it has to be this way but I cannot see an >>> alternative. >> I checked in the patch as is. Using only one separator character for all states >> is just a beautification. > That is not what I was meaning. I would like RTEMS master to use only '-' and > the tester has backward compatibility. I would change everything to '_' separators since we have also the RTEMS options:Dashes are often used as separators so can be questionable to use in strings intended to be a single token.
Ok, so change "EXPECTED-PASS" to "EXPECTED_PASS", all "-" to "_" in report.py, and accept [-_] in report.py.
-- 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
