SetupShibokenAndPyside macro checks module include path and eventually decides to disable respective module in case its include directory is missing. Make this process more straightforward by testing directory existence; "Location: " string is 10 not 9 characters long and leading whitespace makes testing for directory name fail. While there, rename variables to respect that find_pip_package returns only single include and library path.
This folder will contain an individual cmake file for each FreeCAD "helper".
A "helper" should be a macro or function that tries (as much as possible) to adhere to the UNIX philosophy - in other words, it should strive to do one thing and do it well.
The idea here is to break up the cmake build system into smaller, more manageable chunks. This should make maintenance easier, and should also make troubleshooting a bit less painful. Finally, it should also clean up our top-level CMakeLists.txt file a bit, making it bit easier for new developers to jump in and see what's what.
NOTE: CMake has a very distinct difference between its macros and its functions. Namely, a function creates its own SCOPE, whereas a macro essentially executes in the calling scope. There are also some specifics of how arguments are managed.
In general:
- If you use a macro, be careful modifying arguments - you may get surprises.
- If you use a function, make sure to use set(.... PARENT_SCOPE) for variables that should be available outside of the function.