nasa / bplib

Apache License 2.0
27 stars 13 forks source link

Integrate with current OSAL version #165

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

The latest OSAL mainline has support for condition variables and therefore should support all BPLib requirements of the OS.

The embedded "os" calls at https://github.com/nasa/bplib/blob/main/inc/bplib_os.h should be mapped to the corresponding OSAL function. This would largely replace/supplant the existing cfe.c implementation.

The direct POSIX mapping in posix.c could be retained or removed. If removed, there would be a hard dependency on OSAL and all test programs would also have to be built and linked with OSAL. If preserved, it could simplify certain functional test workflows as the software could then be built on Linux with no external dependencies required. However, this could only simplify certain functional/black box tests. For coverage testing, the OSAL-based build would be required as this provides UT assert as well as coverage stub frameworks.