[PATCH rtems-libbsd] waf_libbsd: Account for library check results

[Chris Johns]

hi Kinsey,

This change looks OK to me. Nice work.

Thanks
Chris

On 6/10/21 7:55 am, Kinsey Moore wrote:
> Conditionally compiled tests (consisting only of debugger01) were not
> compiling as expected when libdebugger was present. This appears to have
> occurred during the transition from header detection or due to an
> intervening change in the waf internal libraries. The result of
> check_cc() is the only location this information is reported, so library
> checks now set HAVE_<LIBRARY> as appropriate when the library is found
> so existing code for header configuration correctly recognize that a
> library is present.
> ---
>  waf_libbsd.py | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/waf_libbsd.py b/waf_libbsd.py
> index 97e6d6ee..613797aa 100644
> --- a/waf_libbsd.py
> +++ b/waf_libbsd.py
> @@ -180,10 +180,11 @@ class Builder(builder.ModuleManager):
>                                         mandatory=False)
>                      elif configTest == 'library':
>                          for l in self.data['configure'][configTest][cfg]:
> -                            conf.check_cc(lib=l,
> -                                          fragment=rtems.test_application(),
> -                                          execute=False,
> -                                          mandatory=False)
> +                            if conf.check_cc(lib=l,
> +                                             fragment=rtems.test_application(),
> +                                             execute=False,
> +                                             mandatory=False)
> +                                conf.env['HAVE_%s' % l.upper()] = True
>                      else:
>                          bld.fatal('invalid config test: %s' % (configTest))
>              section_flags = ["-fdata-sections", "-ffunction-sections"]
>