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

Upgrade Ignition versions #111

Merged
merged 6 commits into from May 4, 2021
Merged

Conversation

chapulina
Copy link
Contributor

Edifice packages like ign-msgs7 are failing to install because they require higher versions of packages that are shared with Citadel, like ign-math6.

This PR updates all Citadel packages to their latest versions.

Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina requested a review from j-rivero April 27, 2021 21:24
@chapulina
Copy link
Contributor Author

CI is complaining about missing source packages, is there something else I need to add, @j-rivero ?

@j-rivero
Copy link
Contributor

CI is complaining about missing source packages, is there something else I need to add, @j-rivero ?

After a bit of fighting with that terrible syntax we have it ready I think

@chapulina
Copy link
Contributor Author

Can we get this in, @nuclearsandwich ?

@nuclearsandwich nuclearsandwich self-requested a review May 4, 2021 00:12
@nuclearsandwich
Copy link
Contributor

CI is complaining about missing source packages

Yay the system works!

@nuclearsandwich
Copy link
Contributor

Can we get this in, @nuclearsandwich ?

Thanks for the ping, that or a review request is a great way to bump a request if I don't see it right away. I've made an appointment with myself to perform this import tomorrow at 13:30 Pacific US time.

@nuclearsandwich nuclearsandwich merged commit a2e5eb2 into master May 4, 2021
@nuclearsandwich nuclearsandwich deleted the chapulina/ignition_updates branch May 4, 2021 20:05
@nuclearsandwich
Copy link
Contributor

Import jobs

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.

None yet

3 participants