<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 4, 2019, 3:57 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 04/12/2019 09:49, Chris Johns wrote:<br>
> On 4/12/19 5:43 pm, Sebastian Huber wrote:<br>
>> On 04/12/2019 05:14, Chris Johns wrote:<br>
>>> On 3/12/19 9:59 pm, Sebastian Huber wrote:<br>
>>>> On 02/12/2019 23:32, Chris Johns wrote:<br>
>>>>> On 2/12/19 6:24 pm, Sebastian Huber wrote:<br>
>>>>>> Use the value expected by the RTEMS Tester.  There are two separator<br>
>>>>>> characters used ('-' and '_') for the states.  This is a bit<br>
>>>>>> inconsistent.<br>
>>>>> Sorry about this. I had not noticed.<br>
>>>>><br>
>>>>>> ---<br>
>>>>>>     cpukit/libtest/testbeginend.c | 2 +-<br>
>>>>>>     1 file changed, 1 insertion(+), 1 deletion(-)<br>
>>>>>><br>
>>>>>> diff --git a/cpukit/libtest/testbeginend.c b/cpukit/libtest/testbeginend.c<br>
>>>>>> index b739e47b2e..9bdc86780c 100644<br>
>>>>>> --- a/cpukit/libtest/testbeginend.c<br>
>>>>>> +++ b/cpukit/libtest/testbeginend.c<br>
>>>>>> @@ -29,7 +29,7 @@ rtems_printer rtems_test_printer = {<br>
>>>>>>     static const char* const test_state_strings[] =<br>
>>>>>>     {<br>
>>>>>>       "EXPECTED-PASS",<br>
>>>>> Should this change or should `USER_INPUT` be changed and we update ...<br>
>>>>><br>
>>>>> <a href="https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n159" rel="noreferrer noreferrer" target="_blank">https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n159</a><br>
>>>>> <a href="https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n161" rel="noreferrer noreferrer" target="_blank">https://git.rtems.org/rtems-tools/tree/tester/rt/report.py#n161</a><br>
>>>>><br>
>>>>> ?<br>
>>>>><br>
>>>>> It would be a simpler change because this change means we move to having 2 with<br>
>>>>> `_` and a bunch with `-`. There are internal states in the tester such as<br>
>>>>> `WRONG-VERSION` and changing all of those is a bigger change in the tester.<br>
>>>> If we update the tester, then it should accept '-' and '_' versions so that it<br>
>>>> can be used for test runs before and after a change of<br>
>>>> cpukit/libtest/testbeginend.c.<br>
>>> Yes this is a good point. Shame it has to be this way but I cannot see an<br>
>>> alternative.<br>
>> I checked in the patch as is. Using only one separator character for all states<br>
>> is just a beautification.<br>
> That is not what I was meaning. I would like RTEMS master to use only '-' and<br>
> the tester has backward compatibility.<br>
<br>
I would change everything to '_' separators since we have also the RTEMS <br>
options:<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Dashes are often used as separators so can be questionable to use in strings intended to be a single token.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
*** BEGIN OF TEST SMPMIGRATION 2 ***<br>
*** TEST VERSION: 5.0.0.a6879a418dee37c31728af0d2491cb8924f23de3<br>
*** TEST STATE: EXPECTED-PASS<br>
*** TEST BUILD: RTEMS_NETWORKING RTEMS_POSIX_API RTEMS_SMP<br>
*** TEST TOOLS: 7.5.0 20191114 (RTEMS 5, RSB <br>
f3b44b25d1353253d1454abc695621aafb8c4a97, Newlib d14714c69)<br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div></div>