-
-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
mesa: add license #59231
mesa: add license #59231
Conversation
I think we should hold off on the license based on the discussion in #58225 (comment) |
Yeah, I read about it (I understand lots of the formulae additions are not perfect). But dealing with the license situation one by one can potentially mitigate the issue, either way, our goal is to reach to a place that we can have licenses for every formulae. With that being said, I think this license addition is clear and good from my perspective. |
72e84e9
to
792412e
Compare
I agree, but I think we just aren't ready to start that yet. If you disagree, can you bring it up in #58225 so we can get the opinions of other maintainers who have been involved? |
I don't think anyone suggested stopping all work on licenses. If we're sure about a license and it can be represented by the current implementation I don't see a reason to hold of. |
Okay, this sounds like it was a misunderstanding on my part of what was said. My apologies! CI seems broken though... |
I think it needs a rebase in order to trigger the |
cancelled my PR build. I think the |
792412e
to
ec3bd7c
Compare
Those tests aren't hanging - they're yellow due to the branch protection checks. See #59208 (comment) |
|
add license for mesa project.
license doc ref, https://gitlab.freedesktop.org/mesa/mesa/-/blob/master/docs/license.rst
relates to #59220