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

ignition-transport9: new formula #1065

Merged
merged 4 commits into from
Jul 17, 2020
Merged

ignition-transport9: new formula #1065

merged 4 commits into from
Jul 17, 2020

Conversation

chapulina
Copy link
Contributor

Remove alias and add new formula that depends on ign-msgs6. I took the last commit of gazebosim/gz-transport#149

This is broken off #1063 so we can make the changes incrementally.

Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
Copy link
Contributor Author

CI is failing asking for a license. I didn't find any license information on any formula here, is this a new requirement? @scpeters , please advise.

==> brew audit osrf/simulation/ignition-transport9 --online --new-formula
==> FAILED
Error: 1 problem in 1 formula detected
osrf/simulation/ignition-transport9:
  * No license specified for package.

@chapulina
Copy link
Contributor Author

Merging!

@chapulina chapulina merged commit bb4ae62 into master Jul 17, 2020
@chapulina chapulina deleted the chapulina/dome/transport branch July 17, 2020 20:45
@scpeters
Copy link
Member

CI is failing asking for a license. I didn't find any license information on any formula here, is this a new requirement?

sorry I missed this earlier

yes, this is a new requirement, but only for new formulae. I'll update the existing formulae

@scpeters
Copy link
Member

also, when adding new formulae, we can remove the old alias, but it's also a good time to just add a new alias for the next major version. I'll take care of it for this formula

@scpeters
Copy link
Member

also, when adding new formulae, we can remove the old alias, but it's also a good time to just add a new alias for the next major version. I'll take care of it for this formula

a08e960

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.

3 participants