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

Build from source ign-dome dependencies #232

Merged
merged 8 commits into from
Jul 10, 2020
Merged

Build from source ign-dome dependencies #232

merged 8 commits into from
Jul 10, 2020

Conversation

j-rivero
Copy link
Contributor

Part of the work in gazebo-tooling/gazebodistro#5

@j-rivero
Copy link
Contributor Author

I'm going to merge this so CI from changes in repositories can be tested as well.

@chapulina
Copy link
Contributor

chapulina commented Jul 15, 2020

@j-rivero , note that CI is currently broken for the branches impacted by this PR, because the find_packages haven't been updated yet, see gazebosim/gz-sim#244 for example.

If it will still take some time to push all the changes, maybe we should revert this?

@j-rivero
Copy link
Contributor Author

If it will still take some time to push all the changes, maybe we should revert this?

Part of these changes should be reverted until we have the nightlies in place, I'll do it.

@j-rivero
Copy link
Contributor Author

Part of these changes should be reverted until we have the nightlies in place, I'll do it.

Instead of reverting, let's see if using prerelease repository can fix problems at least with sdf10. gazebo-tooling/gzdev#14

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.

2 participants