-
-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
Proposal: get rid of license
#16692
Comments
No strong opinion on the matter - we could compromise by just making it optional, but that leads quickly to the path of decay and uselessness, so a firm decision one way or another is better. |
I actually really like the idea of removing the license stanza. Having those just makes the cask maintainment more complicated and there is no gain of having those. So I'm good with proposal of removing them. |
Came here to defend the stanza but couldn't come up with a reasonable use case.[1] I agree wholeheartedly that it was weird for it to be required, especially with the
1: It's helpful for users auditing casks and likely very beneficial for users maintaining large systems with specific requirements, but the former doesn't really happen and the latter is really specific and not something they'd rely 100% on hbc for. |
@Amorymeltzer To address your points:
|
What about fonts, though? I don't bother with them, but is this something that never really materialized to an issue on that front, or...? |
Since That’s another big issue of We also were always straightforward about licenses being simply a guideline (i.e. “we’re not lawyers and make no promises to the accuracy of these, you should still check them for yourself”), and not intended to be accurate to the point of minutia (different GPL licenses, despite having so widely different versions they fiercely divide proponents in different camps, are all bundled in |
Indeed. Much as I may enjoy |
Going to change my stance on this - license is nice, and it doesn't really kill us to have in terms of overhead. I would propose making it optional, though, to account for user taps, and then just encourage it to be filled out in Casks submitted to the main repo (as is already done with the comment, no need for more changes there). |
Speaking strictly for myself, yes it does. To me personally, it is a major pain. And I do mean major. It’s not fun going to a repo of something I’ve never heard of and have no interest in using to grill the developer on the license. Or asking a user “is this a free app that works indefinitely but with limitations that can be removed by paying?” (freemium apps are frequently only gratis), or having to check an As for making it optional and encourage it on the main repo, I’m strongly against that. It’d be akin to making I say we either keep it as is, or get rid of it. Compromising is not a good solution in this case. |
#17155, #17156, #17157, #17158 have examples of the (as I see it, useless and harmful) back and forth |
Another point against @jawshooah Do you have an opinion on this issue as a whole? |
Sorry, haven't responded so far as I have no strong feelings one way or the other. I do see the value in removing them though, and not much in keeping them around. As long as we just deprecate the core DSL methods associated with |
Closing in favour of #17437. |
Because the Homebrew Cask maintainers like to do it. See also Homebrew/homebrew-cask#16692
* More on deprecation: Homebrew/homebrew-cask#16692 * This gets rid of the following warning when doing `brew cask info mozart2`: Warning: Calling Hbc::DSL#license is deprecated! There is no replacement. /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:31:in `block in load'
The
license
stanza was thought out in the context of fonts and is nothing but a curiosity for apps (especially we not being about discoverability, and all).license
is (at the time of this writing) missing from just 284 casks in the main repo (from a total of 2937).Rationally, though, there’s no context in which we currently use its information, nor is any real context really planned, nor is anyone eager to do it.
license
is just there, it exists. However, I argue thelicense
stanza is worse than useless, it’s actively harmful. For one, it is required, which makes absolutely no sense for user taps (which I believe we should encourage and have many times been vocal in thinking it is one of the best features of homebrew), but it also delays submissions, as users not filling it up/filling it up wrongly it’s a frequent point of friction for contributions.Pinging @caskroom/maintainers, and I’ll specifically ping @victorpopkov and @Amorymeltzer as they both did a bunch of these.
The text was updated successfully, but these errors were encountered: