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

Add snap packaging #83

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add snap packaging #83

wants to merge 3 commits into from

Conversation

artivis
Copy link

@artivis artivis commented Aug 24, 2021

This PR adds snap packaging to rosboard.

To give it a shot, first install it:

snap install rosboard --channel=edge

And off you go:

rosboard

You don't even need ROS to be installed on the host to use it since the snap packages all rosboard dependencies.

At the moment I own the 'rosboard' name on the snap store; I'd be more than happy to transfer it to you.

Please, feel free to ping me with any question you might have.

A few notes:

  • The snap currently on the store was packaged based on Noetic. Altho I tested it fine against my Melodic simulation.
  • I've added every dependency that explicitly appears in the codebase - whether they are optional or not - to the package.xml file so that e.g. rosdep can install them.
  • Snap being confined, it cannot access ROS packages on the host machine. Therefore you need to bundle every message that you might want to support. For instance, I've added a couple more typical ROS msgs here. One could grow that list based on user requests for instance.
  • The dmesg visualizer is 'disabled' at the moment

Marked as draft until canonical/snapcraft#3570 gets merged.
Marking the PR as ready since canonical/snapcraft#3570 is merged and released on the 'edge' channel.

@artivis artivis marked this pull request as ready for review September 6, 2021 19:20
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.

1 participant