Refactored skill framework and added new enum classes so that a skill can not return idle or running as terminating value

skill provider usability updates. Made virtual methods of skills private and added public non-virtual entry methods to call skills

SickLaserUnit: using cmake target instead of custom mechanism

See merge request ArmarX/RobotAPI!280

removing add_definitions: its affecting all targets. now, it's being set via the imported target sick_scan_base::sick_scan_generic

Fix ukfm static dimension

See merge request ArmarX/RobotAPI!279

# Conflicts:
#	source/RobotAPI/libraries/armem_gui/MemoryViewer.cpp

fixed ltm enable via GUI (marshalling exception on release builds) and commented out locks from memory viewer