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

Update multi-licensed formulae #59467

Closed
wants to merge 23 commits into from

Conversation

dtrodrigues
Copy link
Member

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Update formulae with multiple licenses that were waiting for the next brew release.

@dtrodrigues dtrodrigues added the CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. label Aug 11, 2020
@chenrui333
Copy link
Member

Could be duplicate of #58869

@dtrodrigues
Copy link
Member Author

Good catch, I think almost all except for 4 are in that PR, I'll comment there.

@chenrui333
Copy link
Member

Good catch, I think almost all except for 4 are in that PR, I'll comment there.

Sounds good, you can also push the changes to his PR :)

I think we are in the position to get this out soonish.

@dtrodrigues dtrodrigues deleted the multi-license branch October 17, 2020 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants