Skip to content

Adds mac build support#79

Open
weepeefa wants to merge 2 commits intomainfrom
carlyle/mac_support
Open

Adds mac build support#79
weepeefa wants to merge 2 commits intomainfrom
carlyle/mac_support

Conversation

@weepeefa
Copy link
Contributor

No description provided.

@weepeefa weepeefa requested a review from KipHamiltons June 19, 2021 01:51
Copy link
Member

@KipHamiltons KipHamiltons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ ./b build
[4/10] Linking CXX executable ../controllers/teleport_master_controller/teleport_master_controller
/usr/bin/ld: warning: librobotis-op2.so, needed by /usr/local/webots/projects/robots/robotis/darwin-op/libraries/managers/libmanagers.so, not found (try using -rpath or -rpath-link)
[6/10] Linking CXX executable ../controllers/teleport_controller/teleport_controller
/usr/bin/ld: warning: librobotis-op2.so, needed by /usr/local/webots/projects/robots/robotis/darwin-op/libraries/managers/libmanagers.so, not found (try using -rpath or -rpath-link)
[8/10] Linking CXX executable ../controllers/nugus_controller/nugus_controller
/usr/bin/ld: warning: librobotis-op2.so, needed by /usr/local/webots/projects/robots/robotis/darwin-op/libraries/managers/libmanagers.so, not found (try using -rpath or -rpath-link)
[10/10] Linking CXX executable ../controllers/collect_vision_data/collect_vision_data
/usr/bin/ld: warning: librobotis-op2.so, needed by /usr/local/webots/projects/robots/robotis/darwin-op/libraries/managers/libmanagers.so, not found (try using -rpath or -rpath-link)

This breaks my build on linux. That will need to be fixed

teleport_controller
PRIVATE "${WEBOTS_HOME}/projects/robots/robotis/darwin-op/libraries/robotis-op2/robotis/Framework/include"
)
#target_include_directories(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are these commented?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants