Skip to content

update CMakeLists.txt for Windows build environment#265

Merged
mjcarroll merged 7 commits intoros-perception:melodicfrom
kejxu:export_runtime_binaries_on_windows
Mar 28, 2020
Merged

update CMakeLists.txt for Windows build environment#265
mjcarroll merged 7 commits intoros-perception:melodicfrom
kejxu:export_runtime_binaries_on_windows

Conversation

@kejxu
Copy link
Copy Markdown

@kejxu kejxu commented Feb 2, 2019

update CMakeLists.txt for Windows build environment

  • RUNTIME_OUTPUT_DIRECTORY is needed so that .dll would be exported
  • add if(MSVC) special handling
  • RUNTIME DESTINATION should be ${CATKIN_GLOBAL_BIN_DESTINATION} for .dll on Windows based on catkin recommendation

@mjcarroll
Copy link
Copy Markdown
Contributor

@kejxu or @seanyen Same here with the rebase.

@kejxu kejxu force-pushed the export_runtime_binaries_on_windows branch from 5a09095 to 04d4135 Compare July 16, 2019 18:33
@kejxu
Copy link
Copy Markdown
Author

kejxu commented Jul 16, 2019

@mjcarroll seems like travis is working now =)

@seanyen
Copy link
Copy Markdown
Contributor

seanyen commented Dec 13, 2019

@mjcarroll Just a friendly ping, this is ready for review and merge.

@seanyen
Copy link
Copy Markdown
Contributor

seanyen commented Feb 27, 2020

@mjcarroll Friendly ping.

@mjcarroll mjcarroll merged commit 578af4d into ros-perception:melodic Mar 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants