[PATCH] build: Add "family/" prefix to BSP familiy enable

Chris Johns chrisj at rtems.org
Thu Jul 22 08:47:57 UTC 2021


On 22/7/21 6:37 pm, Sebastian Huber wrote:
> On 22/07/2021 10:33, Chris Johns wrote:
>>>> and so the arch part is not
>>>> really needed. My concern is this type code ...
>>>>
>>>> https://git.rtems.org/rtems_waf/tree/rtems.py#n758
>>>>
>>>> that breaks. Is this an issue? I think a single `/` in a BSP or family is
>>>> cleaner.
>>> Why is this an issue? This BSP family stuff is local to the RTEMS build system.
>> Currently. It is about the symmetry of the naming and how it would look from
>> outside. Nothing more.
> 
> You mean that maybe someone wants to build an application or library for a BSP
> family? I guess this is currently not supported, but you could do this with the
> "bsps/powerpc/motorola_powerpc" approach.

Yes it could happen and this is where the symmetry and the existing code
matters. For example with `bsps/motorola_powerpc` the code can be easily or
cleanly extended by looking for `bsps` as an arch and knowing that is a
`family`. Otherwise the error is `invalid arch/bsp` and then you need check two
lengths etc etc.

Chris


More information about the devel mailing list