Skip to content

QSTAG#196

Merged
gatsoulis merged 5 commits into
strands-project:masterfrom
PDuckworth:qstag
Oct 1, 2015
Merged

QSTAG#196
gatsoulis merged 5 commits into
strands-project:masterfrom
PDuckworth:qstag

Conversation

@PDuckworth

Copy link
Copy Markdown
Member

I've re-tested some simple cases since the documentation was added.

Paul Duckworth added 3 commits September 23, 2015 11:55
@gatsoulis

Copy link
Copy Markdown
Contributor

OK, I ll test it when I have time unless anyone else wants to do it.

@gatsoulis

Copy link
Copy Markdown
Contributor

-_-

@gatsoulis

Copy link
Copy Markdown
Contributor

@marc-hanheide this PR requires igraph package. Any chance of getting it installed on Jenkins?

@gatsoulis

Copy link
Copy Markdown
Contributor

This works fine from what I can tell.

But cannot merge until igraph is on jenkins. For 14.04 it is on the repos as python-igraph package. @marc-hanheide whenever you have a moment would it be possible to apt-get install this package?

@cdondrup

cdondrup commented Oct 1, 2015

Copy link
Copy Markdown
Member

It has to be a rosdep. You can add it here: https://github.com/strands-project/rosdistro/blob/strands-devel/rosdep/strands.yaml if it is just a debian package. Then include it in your package.xml.

EDIT: Jenkins installs every dependency in the package.xml before building it. If it is not in there it will never work.

@gatsoulis

Copy link
Copy Markdown
Contributor

OK great thanks.

Including in the package.xml with the the build_depend and run_depend will suffice?

@cdondrup

cdondrup commented Oct 1, 2015

Copy link
Copy Markdown
Member

@gatsoulis

Copy link
Copy Markdown
Contributor

Looking on this you dont have ghmm as build_depend just run_depend. Will give that a shot. Thanks.

Yiannis Gatsoulis and others added 2 commits October 1, 2015 15:17
gatsoulis pushed a commit that referenced this pull request Oct 1, 2015
@gatsoulis
gatsoulis merged commit c313a68 into strands-project:master Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants