Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

readme: add acceptable casks #1504

Merged
merged 1 commit into from
Dec 20, 2015
Merged

readme: add acceptable casks #1504

merged 1 commit into from
Dec 20, 2015

Conversation

vitorgalvao
Copy link
Member

No description provided.


For this repo, rules for consideration are (following our [nomenclature](https://github.com/caskroom/homebrew-cask/blob/master/CONTRIBUTING.md#finding-a-home-for-your-cask)):

+ Include the latest minor version of legacy versions of commercial and freemium software<sup>1</sup>.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where's the footnote?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forgot about that. Initially was going to include it, but then decided against it. Removed the marker.

@bdhess
Copy link
Contributor

bdhess commented Dec 20, 2015

Just in general, and perhaps the question is tangential to this PR, how do you feel about something like Popularity Contest and developing minimum usage targets to decide when to retire a cask rather than hard limits on number of previous versions or duration for which a Cask should be maintained? That said, I appreciate that you are trying to clarify rules and move things forward immediately, and I'm not trying to stand in the way of that.

@vitorgalvao
Copy link
Member Author

In general, I’m against collecting user data1. Reasons being collected data can be done in two ways, opt-in or opt-out. If it is done by opt-out, it is a privacy violation (however small), but if it is done opt-in, the data is pretty much useless. The solution is, then, to not waste time implementing that.

Having said that, those arguments are in relation to a popularity contest in the sense of “which apps are more popular”. In the case you mention, it’s just about “is anyone even downloading this app”, which I could see a better case for. Even if a few number of users opts-in and we ignore apps that don’t have “hits”2, we could still consider the ones that have them and would be removed otherwise, as apps we should keep.

I’m still not too comfortable with the idea, though. How would we do it? Once you have homebrew-cask installed, you don’t need to have any more contact with us. Everything happens on your machine, and you call the app servers directly when you need to install something.

This would require we either keep a list of everything you installed and periodically send it to us (yuck! Also, goes against our new mode of thinking where we try to emulate a manual installation as closely as possible) or that we, for instance, send a hit to a server somewhere whenever you run a brew cask install.

All of those require we maintain and pay for a server somewhere for this (not to mention build and maintain the code for this in homebrew-cask itself) and I simply cannot see how it’d be worth it, particularly right now when we have real functional issues to solve.

Lastly, since we’ll be joining up with homebrew, this also seems like something weird to have when they wouldn’t, particularly since we’ll try to reuse their code as much as we can. For this to be implemented, seems like it would need to be a discussion and implemented on the core.


1 Not recommending you read through the issue, though, just giving it as context in case you do want to read my points.

2 As in “were we hit with a request for it”.

@adidalal
Copy link
Contributor

LGTM.

vitorgalvao added a commit that referenced this pull request Dec 20, 2015
@vitorgalvao vitorgalvao merged commit b056393 into master Dec 20, 2015
@vitorgalvao vitorgalvao deleted the readme-acceptable branch December 20, 2015 16:59
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants