Changes to allow for support on macOS

What

This MR makes a few changes and fixes to allow for compiling on macOS.

All checks are done on my laptop, running macOS Catalina v10.15.7. The changes should not really depend on the explicit macOS, however.

Remove Linux-specific sources from the build

The main change is to the CMakeLists.txt file in libDevCom. There a new list of sources are created which contain the source for devices that depend explicitly on Linux drivers. If, during the CMake configuration step it is detected that we are on macOS, those sources are not included in the build.

The current sources that are found to depend explicitly on Linux device drivers are: SPIDevCom and I2CDevCom.

Fix the check for Clang

Updates the check for Clang version by using the MATCHES requirement instead of the STREQUAL requirement in src/CMakeLists.txt:

if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")

becomes

if ("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")

Miscellaneous

Change CMake message outputs from message(...) to message(STATUS...)

added bug enhancement labels

changed the description

added 1 commit

• b0c596f3 - fix labRemote python import for python3

Compare with previous version

Hi @dantrim ,

I think it might be better to not compile Linux-specific classes on Mac. We won't have to remember to add dummy functions everytime.

ie: only add to the sources list if on Linux.

Yeah that was my first inclination, but then I saw what was being done already in I2CDevCom. I'll give it a try and update things.

added 1 commit

• fa91edc9 - revert back to relative import

Compare with previous version

added 2 commits

• 270c9b48 - remove linux specific devices from source list of running on mac
• 237fadb3 - Merge branch 'dantrim_mac_support' of...

Compare with previous version

added 1 commit

• 388e8333 - message status, and check for CMAKE_SYSTEM_NAME

Compare with previous version

resolved all threads

added 1 commit

• 0121fba3 - comment and revert input-hw

Compare with previous version

changed the description

mentioned in issue #56 (closed)

approved this merge request

Hi @epianori is this able to be merged?

mentioned in commit 4b72c35c

merged