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

Mention homebrew-binary in contributing docs #4228

Merged
merged 1 commit into from
May 26, 2014
Merged

Mention homebrew-binary in contributing docs #4228

merged 1 commit into from
May 26, 2014

Conversation

vitorgalvao
Copy link
Member

There have been a lot of cli-only app submissions, recently, and it’s clear that few know of homebrew-binary. As suggested, this documents and clarifies cases when a cask should be submitted first to hombrew-binary.

@tapeinosyne
Copy link
Contributor

Homebrew-binary seems significantly less active than Cask; furthermore, the Cask DSL works rather well with naked binaries.

In practice, would it be desirable to divert binary-only casks?

@rolandwalker
Copy link
Contributor

+1 for merge as this doc reflects current practice and saves typing.

However, I agree with @ndr-qef that we should consider revisiting this policy in the future.

One other point: I don't think homebrew-binary accepts pkg binaries; we have always accepted those even when CLI-only.

@alebcay
Copy link
Member

alebcay commented May 6, 2014

That would be correct, @rolandwalker, homebrew-binary only accepts archive files (zip, etc.), but not pkg installers. In fact, I don't think they even take disk images (the Formula DSL doesn't support mounting images, I think).

@nanoxd
Copy link
Contributor

nanoxd commented May 6, 2014

👍

@vitorgalvao
Copy link
Member Author

At the moment we have an open pull request for sketchtool, which is a cli-only app. It was submitted to homebrew and rejected, but what’s interesting is two owners (@adamv and @jacknagel) suggested homebrew-cask and not homebrew-binary, as the place for submission.

With that in mind, and considering this repo is more active and supports .pkg and .dmg, perhaps we should propose substituting homebrew-binary with homebrew-cask. Duplicating work is inefficient, and dividing submissions between projects depending on support for handling the downloaded file is confusing.

@rolandwalker
Copy link
Contributor

Another point in favor of CLI is that we have built up a good rhythm and should have no problem undertaking the small additional load.

@phinze
Copy link
Contributor

phinze commented May 11, 2014

In the past my general strategy for inclusion of any app was basically "Sure - everybody's welcome!"

More recently I've encountered a few users who have been confused about what sort of apps they can expect to exist in the Cask repo, which caused me to reconsider my stance. I wondered if we should make a clearer definition of what "belongs" in our repo.

But I'm still not really sold on either side. Sounds like the mood here is to make the default policy be for inclusion? If that's the consensus I'd be okay with that - either way definitely +1 for documenting clearly the policy / process.

@vitorgalvao
Copy link
Member Author

I say we start accepting binary-only submissions. We already have some, and the line of acceptance is ever more blurry (even amongst maintainers, as @ndr-qef pointed out).

A lot of users see homebrew-cask as the right place to submit them, so we can deduce a lot of them will also think of it as the right place to search for them. Since that expectations exists and we’re the best equipped to fulfil it anyway (for reasons previously mentioned, like support for more containers), it makes sense for us to do it.

Regarding homebrew-binary, we can make the suggestion to shift the focus on binaries to homebrew-cask, but that might not be entirely desirable (some users might just want a few of them and not want to use this project to do it, if regular homebrew works fine for them). Naturally, their issue tracker is a better place to find this out.

@rolandwalker
Copy link
Contributor

@vitorgalvao I agree. But then do we need to change the docs at all?

@vitorgalvao
Copy link
Member Author

We needn’t, no. It seems we’re now mostly in favor of inclusion, but just to make sure, @nanoxd and @alebcay, considering the arguments in this issue, do you have an opinion either way?

@nanoxd
Copy link
Contributor

nanoxd commented May 26, 2014

@vitorgalvao I'm fine with that. That would ease managing casks.

@nanoxd nanoxd merged this pull request into Homebrew:master May 26, 2014
@nanoxd
Copy link
Contributor

nanoxd commented May 26, 2014

Accidentally merged this in, I removed it from the repo and kept everything else intact.

@vitorgalvao vitorgalvao deleted the mention-homebrew-binary branch May 26, 2014 17:43
@vitorgalvao vitorgalvao mentioned this pull request Aug 12, 2014
@vitorgalvao
Copy link
Member Author

@caskroom/maintainers There is some confusion (at least on my part) as to what exactly is the rule we agreed on, in this issue.

This is surfacing due to a comment @radeksimko made, that applies (most recently) to cases like packer and terraform (multiple times).

From our discussion in this issue, I do believe these should be included. Limiting the rule to “include binaries only if they come in a package format unsupported by homebrew(/binary)” and/or “if the source is not easily available” is confusing, and makes no sense to me. Why should we be filtering submissions based on how they’re compressed (especially since that’s something that can so easily change), or on the source code’s availability (there are many reasons to prefer the binaries, like not having to wait for it to compile)? Another reason we decided for inclusion was that we’re more active than homebrew/binary and many contributors already naturally gravitate towards us for binaries. We have the infrastructure, and our syntax is more familiar to those contributors, so why go against that expectation, and with a confusing (and nonsensical, I believe) “sometimes it’s ok, sometimes it’s not” rule, on top of that?

Either way, even if we decide to deny those submissions on those grounds, these cases should be clearly defined.

@rolandwalker
Copy link
Contributor

I think the consensus above is clearly in favor of allowing binaries:

  • There is no maintainer against, and several in favor
  • Cask authors want to do it
  • Homebrew wants us to do it, at least in some cases

Miscommunication is understandable though. We didn't change the docs (as it was an unwritten rule) so there's no guarantee that all maintainers followed this lengthy discussion. Not everyone is on IRC either. Perhaps we should have sent around an email. @fanquake also requested email for certain other internal/policy decisions.

@tapeinosyne
Copy link
Contributor

As previously expressed, I think it desirable to merge binary-only packages. Opinions aside, I very much agree with @vitorgalvao in that clarity is a primary concern. (I might be mistaken, but was there not an instance of someone being rejected here, sent to Homebrew-binary, and pinballed back to Cask?)

Tangentially, I wonder if we should approach the maintainers of Homebrew and Homebrew-binary. It would be helpful to reach a consensus on the canonical location of binary packages. (The issue might be murkier than we suspect, as many formulae now use poured bottles which overlap upstream binary distributions.)

(Addendum: I am not very sure about how we could sponsor Cask politely. “Excuse me, sir, I think we are just more popular; could you please drop your project and support ours instead?”)

@vitorgalvao
Copy link
Member Author

I am not very sure about how we could sponsor Cask politely.

The issue would be complicated further if homebrew-cask does eventually become a project independent of homebrew.

On the other hand, even some homebrew owners suggested homebrew-cask for a binary in at least one case.

@adidalal adidalal removed the awaiting maintainer feedback Issue needs response from a maintainer. label Jan 3, 2016
@miccal miccal removed the documentation Issue regarding documentation. label Dec 23, 2016
@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.

8 participants