<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 2, 2020 at 12: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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 01/10/2020 23:38, Joel Sherrill wrote:<br>
<br>
> I am generating a uniquely named ini file per BSP. In doing an ls -l, <br>
> I noticed the size varied by at least a factor of three. This is <br>
> because when some BSPs are put in, the entire family is being <br>
> included.  I suspected this happens when a BSP name matches the family <br>
> name but leon3 doesn't trip this behavior. Here is an example:<br>
><br>
> config-arm-lpc32xx_mzx.ini:[arm/lpc32xx_mzx]<br>
> config-arm-lpc32xx_mzx.ini:[arm/lpc32xx_mzx_stage_1]<br>
> config-arm-lpc32xx_mzx.ini:[arm/lpc32xx_mzx_stage_2]<br>
><br>
> I don't think this behavior is correct. I am asking for the bsp <br>
> defaults for a single BSP variant.<br>
<br>
I guess you refer to the behaviour of:<br>
<br>
     --rtems-bsps=REGEX,...<br>
                         a comma-separated list of Python regular <br>
expressions which select the desired BSP variants (e.g. 'sparc/erc32'); <br>
it may be used in the bsp_defaults and bsp_list commands<br>
<br>
It is currently used like this:<br>
<br>
def is_in_white_list(variant, white_list):<br>
     if not white_list:<br>
         return True<br>
     for pattern in white_list:<br>
         if re.search(pattern, variant):<br>
             return True<br>
     return False<br>
<br>
Maybe we should change it to:<br>
<br>
def is_in_white_list(variant, white_list):<br>
     if not white_list:<br>
         return True<br>
     for pattern in white_list:<br>
         if re.match(pattern + "$", variant):<br>
             return True<br>
     return False<br></blockquote><div><br></div><div>Yes. That would seem to be more right. Should it have a ^ at the front?</div><div>I think we want an exact match versus a partial one.</div><div><br></div><div>--joel</div><div><br></div><div> </div></div></div>