-
Notifications
You must be signed in to change notification settings - Fork 31
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 ikos-underlay and ikos builds. #182
Conversation
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.
I'm not familiar with the details of the YAML file for CI, but LGTM.
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.
I'm not familiar with the details of the YAML file for CI, but LGTM.
bc85a83
to
df2cfa5
Compare
The ikos-underlay job builds [IKOS] (currently from a custom fork) in an underlay by itself. This is done so that its compiler wrappers can be used during the build of a ROS workspace in order to facilitate analysis with the nascent [ament_ikos] package. [IKOS]: https://github.com/NASA-SW-VnV/ikos [ament_ikos]: https://github.com/ament/ament_ikos
Not all of these are strictly runtime dependencies but in trying to get the build running I've just brought all of them forward.
370d605
to
c7b97ba
Compare
Update repos file urls.
b3cdbc5
to
e26714a
Compare
Since we ended up removing these jobs, I'm going to close this PR. I don't think we are going to move forward with this. Just in case, I won't delete the branch for now. |
The ikos-underlay job builds IKOS (currently from a custom fork) in an
underlay by itself.
This is done so that its compiler wrappers can be used during the build
of a ROS workspace in order to facilitate analysis with the nascent
ament_ikos package.