-
Notifications
You must be signed in to change notification settings - Fork 145
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
Tweak Ubuntu source install #136
Conversation
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
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.
small comment in the troubleshooting section
Signed-off-by: Louise Poubel <[email protected]>
I ran the latest tutorial in a Docker container, and the dependencies line seems not very reliable.
When I run just the
I'm not sure why it has all the junk in the beginning. The effect is that the |
Ohh maybe it needs this fix? |
Ah, that does look a lot better:
|
#141 fixes the command |
Ah I forgot to say, there was still some packages not installed after all the above.
It went away after I installed Similarly for
It looks like the command does look for sdf... so I'm not sure how to get this into the list programmatically. |
I believe the idea is to leave the optional tooling packages out, because most users will not want them. If someone wants to run memory leak tests or generate docs, they should install those dependencies separately. |
Then the Doxygen should be a warning, not an error, right? The |
Oh yeah I didn't realize it was an error. It's interesting that CI is not complaining, I wonder what's the difference in the setup. Doxygen version perhaps?
You're right, I think https://github.com/osrf/sdformat/blob/master/.github/ci/packages.apt |
I was setting up a new machine and noticed some places that could use improvements.
packages.apt
(fixes [Ubuntu] When installing from source, installing dependencies also tries to install ignition packages #119)