Skip to content

fix on CMakeFiles to let the package compile in Windows 10 - ROS Noetic #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
enricovillagrossi wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix on CMakeFiles to let the package compile in Windows 10 - ROS Noetic #53

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0354d45
fix on CMakeFiles to let the package compile in Windows 10 - ROS Noetic
enricovillagrossi May 16, 2022
e66d4b1
moved the compile flag -DNOGDI from add_defintions to target_compile_…
enricovillagrossi Dec 14, 2022
f4a45fb
moved the flag -DNOGDI in target_compile_definitions
enricovillagrossi Dec 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions abb_egm_hardware_interface/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,11 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
target_compile_options(${target_name} PRIVATE /W4)
endif()

## Restrict Windows header namespace usage
if (WIN32)
target_compile_definitions(${target_name} PUBLIC -DNOGDI)
endif()

#-----------------------------
# Executable target
#-----------------------------
Expand Down
5 changes: 5 additions & 0 deletions abb_egm_state_controller/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,11 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
target_compile_options(${target_name} PRIVATE /W4)
endif()

## Restrict Windows header namespace usage
if (WIN32)
target_compile_definitions(${target_name} PUBLIC -DNOGDI)
endif()

#===========================================================
# Install
#===========================================================
Expand Down