Skip to content

Commit

Permalink
Fix permissions for installation of OMSimulator exes. (#9670)
Browse files Browse the repository at this point in the history
  - We install them using the DIRECTORY specifier which does not give them
    the needed permissions to execute. Use source permissions explicitly.

    There is still some inconsistency in what permission OMSimulator sets
    for its executables. This should be fixed because it can lead to some
    confusing errors where some executables run fine while others fail.
  • Loading branch information
mahge committed Nov 8, 2022
1 parent f18be57 commit c921f86
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion omsimulator.cmake
Expand Up @@ -92,7 +92,8 @@ else()


install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/OMSimulator/bin/
DESTINATION ${CMAKE_INSTALL_BINDIR})
DESTINATION ${CMAKE_INSTALL_BINDIR}
USE_SOURCE_PERMISSIONS)


install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/OMSimulator/include/
Expand Down

0 comments on commit c921f86

Please sign in to comment.