nasa / bplib

Apache License 2.0
30 stars 13 forks source link

Update Default Build Settings and/or README; #231

Open DavidEdell opened 1 year ago

DavidEdell commented 1 year ago

bplib cannot be built as-is with the instructions in the README due to a missing dependency on OSAL.

After some digging, I was able to build using cmake -DBPLIB_OS_LAYER="POSIX" -DBPLIB_ENABLE_UNIT_TESTS=0 ...

Either the CMake defaults should be updated to make the above settings the default, or that snippet should be added to the README.

It would also be beneficial to add instructions on how to build this library with OSAL. Before finding the above settings, I got cmake to proceed farther with CMAKE_PREFIX_PATH=../../osla/build_osal cmake .. only for it to fail in searching for NasaOsalTargets.cmake.

EDIT: To build the test application, I also had to add to the cmake line -DBPLIB_BUILD_TEST_TOOLS=1 -- the README implies that the test application should be built by default.