Add compiler options (#6)

tue-robotics · Sep 28, 2022 · 12f9801 · 12f9801
2 parents cc44c77 + 6257de2
commit 12f9801
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -1,6 +1,9 @@
 cmake_minimum_required(VERSION 2.8.3)
 project(tue_ethercat_drivers)
 
+add_compile_options(-Wall -Werror=all)
+add_compile_options(-Wextra -Werror=extra)
+
 find_package(catkin REQUIRED 
   soem_master
   roscpp
@@ -10,17 +13,20 @@ find_package(catkin REQUIRED
 find_package(OROCOS-RTT REQUIRED ${RTT_HINTS} )
 include(${OROCOS-RTT_USE_FILE_PATH}/UseOROCOS-RTT.cmake)
 
+find_package(soem REQUIRED)
+
+orocos_use_package(soem_master)
+orocos_use_package(soem_beckhoff_drivers)
+
 include_directories(
+  SYSTEM
   ${catkin_INCLUDE_DIRS}
+  ${soem_INCLUDE_DIRS}
+  ${USE_OROCOS_INCLUDE_DIRS}
 )
 
-find_package(soem REQUIRED)
-include_directories(${soem_INCLUDE_DIRS})
-
 file(GLOB_RECURSE HEADER_FILES include/*.hpp)
 
-orocos_use_package(soem_master)
-
 # TUe EtherCAT print drivers
 orocos_plugin(TUeES020 src/soem_beckhoff_drivers.cpp src/TUeES020.cpp ${HEADER_FILES})
 target_link_libraries(TUeES020 ${soem_LIBRARIES})

diff --git a/src/soem_beckhoff_drivers.cpp b/src/soem_beckhoff_drivers.cpp
@@ -29,7 +29,7 @@
 #include <rtt/plugin/Plugin.hpp>
 
 extern "C" {
-bool loadRTTPlugin(RTT::TaskContext* c){
+bool loadRTTPlugin(RTT::TaskContext* /*c*/){
   return true;
 }
 }