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

Add homebrew #56

Merged
merged 1 commit into from
Apr 28, 2016
Merged

Add homebrew #56

merged 1 commit into from
Apr 28, 2016

Conversation

v9n
Copy link
Contributor

@v9n v9n commented Jan 19, 2016

This allow people to install it via homebrew on Mac OSX like:

brew install sup

We can test this locallly with brew install sup.rb

@pkieltyka
Copy link
Contributor

This is a great idea. Let's give this a test drive and submit to homebrew.

@pkieltyka
Copy link
Contributor

thanks @kureikain


go_resource "gopkg.in/yaml.v2" do
url "https://gopkg.in/yaml.v2.git", :revision => "f7716cbe52baa25d2e9b0d0da546fcf909fc16b4"
end
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we run make tools deps install or go get -u github.com/pressly/sup/cmd/sup instead of pulling all the above dependencies manually?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They prefer to use go_resource

@VojtechVitek
Copy link
Collaborator

@kureikain nice, this will be really useful :)

@@ -0,0 +1,41 @@
require "language/go"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we move this file under release/ directory? One day, when we reach v1.0, we should probably try to open PR against https://github.com/Homebrew/homebrew/tree/master/Library/Formula

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, will update it.

I actually open a PR on homebrew Homebrew/legacy-homebrew#48230.

Later on, we probably should have some automation process to update and send PRs to homebrew use the sup.rb version on this repository as the source.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, nice! So in that case, do we even need the file in this repository too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not really sure if we should keep them in this repository or just rely on homebew and a package maintainer.

However, my opinion is that we should keep this file in release/sup.rb as a source of truth. And later on we may have some make release_brew that automatically checkout and push a branch on the dev account(like pressly/homebrew). Then the dev simply create a PR on homebrew using github UI.

It also help power user who may run: brew install https://raw.githubusercontent.com/pressly/sup/master/release/sup.rb if homebrew doesn't merge quickly for whatever reason.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Alright, let's merge this then (after v0.4 bump). Thanks for explanation.

@eduardonunesp
Copy link
Contributor

Totally agree, awesome idea 👍

@VojtechVitek VojtechVitek merged commit 43d7c5c into pressly:master Apr 28, 2016
@ernoaapa ernoaapa mentioned this pull request Dec 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants