-
Notifications
You must be signed in to change notification settings - Fork 46
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
Build in Ubuntu 22.04/24.04 #173
base: master
Are you sure you want to change the base?
Conversation
I deleted the 50-rosmon.bash and was able to get rosmon working with rosrun:
Another test involving nodelets doesn't work:
https://github.com/lucasw/image_manip/blob/master/image_manip/launch/convert.launch (can retry with more basic nodelet example) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, apart from the Qt change (see comment). We will need to test this on Melodic + Noetic with the Qt versions of the standard Ubuntu versions as well.
Regarding the |
…pace ‘std’ does not name a type' error in C++17
…bash to never return
I saw https://robotics.stackexchange.com/a/24326/884 and thought to try rosmon out, but first I need it to build in 22.04 (using https://github.com/lucasw/ros_from_src/tree/robot_state_publisher/ubuntu_2204).
I'm not sure about the deprecated qt backgroundcolorrole change, the change was suggested by https://doc-snapshots.qt.io/qt5-5.15/qt.html
I haven't been able to test this yet because of an issue with
devel/etc/catkin/profile.d/50-rosmon.bash
, source devel/setup.bash gets locked up.xqms/rosfmt#6 is needed along with this.