On 19/03/2021 14:26, Kinsey Moore wrote:

This was my suggestion since the introduction of the waf build system
preserved the names of the executables generated as compared to the
autotools build, but offered different internal names when setting
expected test states. The waf build system was new and different, so I
thought it better to continue matching existing behavior.

The waf based build system was intended to work with Doorstop. This is one of the reasons why the items are in YAML format. A limitation of Doorstop was that links had no role attribute. This was one of the reasons why I replaced Doorstop with custom modules. The

 actions:
 - set-test-state:
     spconfig01: exclude

is an early hack which was necessary to avoid cyclic dependencies. Since links 
have now a role, we have more flexibility. We could introduce a new link role 
for BSP-specific test program states. These links can be automatically verified 
and also processed.

Also the comments you added to the items contradict the philosophy of the 
structured information in the items. A rationale why a test program has a 
BSP-specific state should be in a dedicated attribute. Custom links can have 
custom attributes for example.

Before we rename hundreds of files we should try to fix the underlining problem.

--
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to