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.

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

Mac OS Binary Installation fail caused by version difference of tinyxml2 #440

Closed
tilt-silvie opened this Issue Dec 23, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@tilt-silvie

tilt-silvie commented Dec 23, 2017

Bug report

Required Info:

  • Operating System:
    • Mac OS 10.12.6
  • Installation type:
    • binaries
  • Version or commit hash:
    • ros2-ardent-package-osx-fastrtps-x86_64.tar.bz2
  • DDS implementation:
    • Fast-RTPS

Steps to reproduce issue

Following instruction below
https://github.com/ros2/ros2/wiki/OSX-Install-Binary

Then, I try to run demo ros2 run demo_nodes_cpp talker, but it failed.

ROS2 required tinyxml 2.5. However, tinyxml which is on brew repository is versioned to 2.6 in 11 Dec 2017.

Workaround

Downgrade tinyxml2 to 5.0.1

cd /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/
git checkout 0b3ee2f67043af1b4270096c13350420e3290399
brew unlink tinyxml2
HOMEBREW_NO_AUTO_UPDATE=1 brew install tinyxml2
git checkout master

@tilt-silvie tilt-silvie changed the title from OSX Binary Installation fail caused by version difference of tinyxml2 to Mac OS Binary Installation fail caused by version difference of tinyxml2 Dec 23, 2017

@mikaelarguedas

This comment has been minimized.

Show comment
Hide comment
@mikaelarguedas

mikaelarguedas Dec 23, 2017

Contributor

Thanks @tilt-silvie for reporting this! I'll update the installation instructions accordingly

Then, I try to run demo ros2 run demo_nodes_cpp talker, but it failed.

Can you provide the error message for book-keeping please?
Error message:

dyld: Library not loaded: /usr/local/opt/tinyxml2/lib/libtinyxml2.5.dylib
  Referenced from: /private/tmp/tinyxml_testing/ros2-osx/./lib/demo_nodes_cpp/talker
  Reason: image not found

Clarification for future readers:

This is not related to the tinyxml package (that is indeed 2.6 in homebrew since it's addition to the homebrew formulaes) but the tinyxml2 package that got bumped from 5.0.1 to 6.0.0 on December11 (Homebrew/homebrew-core@7eaa518)

Workaround:
Remove tinyxml2 if it's installed:

brew uninstall --force tinyxml2

Install and pin the tinyxml2 version

brew install https://raw.githubusercontent.com/Homebrew/homebrew-core/0b3ee2f67043af1b4270096c13350420e3290399/Formula/tinyxml2.rb
brew pin tinyxml2
Contributor

mikaelarguedas commented Dec 23, 2017

Thanks @tilt-silvie for reporting this! I'll update the installation instructions accordingly

Then, I try to run demo ros2 run demo_nodes_cpp talker, but it failed.

Can you provide the error message for book-keeping please?
Error message:

dyld: Library not loaded: /usr/local/opt/tinyxml2/lib/libtinyxml2.5.dylib
  Referenced from: /private/tmp/tinyxml_testing/ros2-osx/./lib/demo_nodes_cpp/talker
  Reason: image not found

Clarification for future readers:

This is not related to the tinyxml package (that is indeed 2.6 in homebrew since it's addition to the homebrew formulaes) but the tinyxml2 package that got bumped from 5.0.1 to 6.0.0 on December11 (Homebrew/homebrew-core@7eaa518)

Workaround:
Remove tinyxml2 if it's installed:

brew uninstall --force tinyxml2

Install and pin the tinyxml2 version

brew install https://raw.githubusercontent.com/Homebrew/homebrew-core/0b3ee2f67043af1b4270096c13350420e3290399/Formula/tinyxml2.rb
brew pin tinyxml2
@mikaelarguedas

This comment has been minimized.

Show comment
Hide comment
@mikaelarguedas

mikaelarguedas Dec 23, 2017

Contributor

Update the error message and confirmed the updated instructions fix the problem, closing this

Contributor

mikaelarguedas commented Dec 23, 2017

Update the error message and confirmed the updated instructions fix the problem, closing this

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