Skip to content
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

Updates for Mojave and Catalina #141

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

Conversation

smnogar
Copy link

@smnogar smnogar commented Oct 14, 2019

I have had pretty good success getting this to build on recent versions of macOS. It has some specific versions of software like boost to increase the chances that this script continues to work.

README.md Outdated
* **opencv3**
* **python2**
* **sip**: 4.19.8_12 - Required for python2 support
* **pyqt**: 5.10.1_1 - Required for python 2 support
Copy link
Owner

Choose a reason for hiding this comment

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

How realistic would it be to have Python 3 as the default now? We're using it for most of the base system at CPR, so I'd expect it to mostly work.

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure. Isn't ROS planning on officially adopting it as of ROS "noetic" which isn't ready yet? I'll play around with it in a couple months before python 2 support officially gets dropped.

smnogar and others added 6 commits October 20, 2019 20:59
* Added a simple patch so that FindFLANN can find the flann library when building pcl_ros.
* Remove pcl_catkin
* Add patch for rospy to get rid of “concurrent poll() invocation" errors
* Build gazebo9 from scratch to help eliminate compatibility issues with protobuf
if the link already exists it stops
smnogar and others added 9 commits February 24, 2020 13:06
upgrade to use python3, cleans out the manual installation of old dependencies that still had python2 support. Manually builds tango icons so that rqt has icons again.
splitting these branches up for the time being
@fjp
Copy link

fjp commented May 10, 2020

Great work @smnogar! I hope this is merged one day and included in the documentation on how to install ros on osx.

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.

4 participants