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

Remove extension test #668

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

Conversation

matt-attack
Copy link
Contributor

@matt-attack matt-attack commented Jan 20, 2020

Not ready yet, just want to be able to run the CI.

Theoretically this fixes the ARM64 builds, or at least the OpenGL issue with them.

@matt-attack
Copy link
Contributor Author

@pjreed how hard would it be to make the CI also build this in ARM64?

@pjreed
Copy link
Contributor

pjreed commented Jan 20, 2020

I've looked into it a little bit, and I don't think it'd be too hard to make the ROS Industrial scripts also run in an ARM64 image. I'll experiment with it.

@pjreed
Copy link
Contributor

pjreed commented Jan 20, 2020

Ok, I think I've gotten the CI to build on ARM64 with both Kinetic and Melodic. Try cherry-picking this into your branch: https://github.com/pjreed/mapviz/tree/add-arm64-ci

@pjreed
Copy link
Contributor

pjreed commented Jan 20, 2020

Drat, job timed out. This should also be running in the PR tester on the ROS build farm, though, so I'm going to experiment and figure why that's not showing up here...

@pjreed
Copy link
Contributor

pjreed commented Jan 20, 2020

The mapviz config in the rosdistro repository was pointed at the kinetic-devel branch rather than master, which means that it's only testing PRs that target kinetic-devel. I made a PR to fix that, but after it gets accepted, we'll have to wait overnight for it to reconfigure the Jenkins job on the build farm.

After that, the official ROS build farm will start building PRs... although it seems like it always builds them for amd64 rather than ARM, so that might not help us.

@pjreed
Copy link
Contributor

pjreed commented Jan 20, 2020

I added another commit to my branch that should work much better; it will make the builds run in native ARM64 build environments instead of qemu. Give that a try.

@matt-attack
Copy link
Contributor Author

@danthony06 were you building on 18.04? Seems like the CI arm build fails in the exact same way on that but works on kinetic.

@danthony06
Copy link
Contributor

danthony06 commented Jan 21, 2020 via email

@pjreed
Copy link
Contributor

pjreed commented Jan 21, 2020

@pjreed
Copy link
Contributor

pjreed commented Jan 23, 2020

Nice, build looks good.

mapviz/src/map_canvas.cpp Outdated Show resolved Hide resolved
@pjreed
Copy link
Contributor

pjreed commented Jan 27, 2020

It looks like the build is now broken on AMD64 platforms: http://build.ros.org/job/Kpr__mapviz__ubuntu_xenial_amd64/9/console

@matt-attack
Copy link
Contributor Author

Should be fixed now

@matt-attack
Copy link
Contributor Author

Guess just needs to be tested on AMD64 to make sure I didnt break the screenshot acceleration

@danthony06
Copy link
Contributor

Looks like it's not working on a real display. When I try to add a point cloud or image, I get the following error on the command line, and mapviz appears to hang.

(mapviz:7969): GLib-GObject-WARNING **: 15:18:14.497: cannot register existing type 'GtkStyle'

(mapviz:7969): GLib-GObject-WARNING **: 15:18:14.497: cannot add private field to invalid (non-instantiatable) type '<invalid>'

(mapviz:7969): GLib-CRITICAL **: 15:18:14.497: g_once_init_leave: assertion 'result != 0' failed

(mapviz:7969): Gtk-CRITICAL **: 15:18:14.497: gtk_paint_vline: assertion 'GTK_IS_STYLE (style)' failed

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