uniba-swt / libbidib

A library for communication with a BiDiB (www.bidib.org) system using a serial connection.
GNU General Public License v3.0
10 stars 4 forks source link

CMake improvements and PKG-Config Integration #12

Closed BLuedtke closed 2 years ago

BLuedtke commented 2 years ago

Closes #11

CMakeLists.txt now provides an install target. This installs the dynamic and static library, as well as the include headers. These are installed according to the system defaults. On Linux, this is /usr/local/lib and /usr/local/include respectively. Furthermore, a pkg-config (.pc) is installed for each library version respectively. This makes the libraries easily includable from other CMakeLists.txt.

The above functionality was added in the first commit. The most changes in CMakeLists.txt in the second commit are simply formatting.

I have not been able to test this on a Mac. Not sure what the default paths are there, but should work nonetheless.

BLuedtke commented 2 years ago

Ah, tests for bidib_static are not running fully. Let me fix that first.

eyip002 commented 2 years ago
BLuedtke commented 2 years ago

Thank you for making the macOS adjustments.

Has this been tested on one of the SWTbahn platforms?

Good point - not yet. No change to actual source so should run the same, but definitely better to make sure that the CMake version on the raspberri's is new enough. I'll do that once I'm healthy again.

Does relevant portions of the README file need to be updated?

A little bit, I guess. I'll do so once I've tested how it works on the Pi's

BLuedtke commented 2 years ago

Thinking about the CMake Setup - If we provide an install target, it's probably a good idea to also provide an uninstall target. These are a little more complicated to setup, but I shall have a look at how it can be done here.

eyip002 commented 2 years ago

Tested on SWTbahn Standard