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

Build and Publish Homebrew Package #330

Merged
merged 4 commits into from
Apr 15, 2022

Conversation

drewstinnett
Copy link
Contributor

This will add a homebrew targets in, using the existing goreleaser process. This PR should address #186

Note that I've got my personal repo in there now as a target (mainly so I could test that it actually works). These should be changed to whatever you folks think is appropriate 😁

@CLAassistant
Copy link

CLAassistant commented Apr 8, 2022

CLA assistant check
All committers have signed the CLA.

@dustin-decker
Copy link
Contributor

Thanks a lot, Drew! Looking forward to getting this released

@drewstinnett
Copy link
Contributor Author

Happy to help however I can!

@dustin-decker
Copy link
Contributor

Do you know if there is any downside to having the brew formulas in this repository?

@drewstinnett
Copy link
Contributor Author

I don't think it can go directly in this repository...in order for the tap names to work, the repository name needs to be prefixed with homebrew-. So for example, it would need to go into a repository like:

github.com/trufflesecurity/homebrew-trufflehog

Then it could be tapped with: brew tap trufflesecurity/trufflehog (Note how the homebrew- is dropped from the tap cli)

What some folks do, is create a tap to hold all their formulas, which may be good here if there are other homebrew packages you wanna put up. In this case, you could have something like trufflesecurity/homebrew-tap, which would work with brew tap trufflesecurity/tap

Full docs on this is here, if it's helpful!

@dustin-decker
Copy link
Contributor

Thanks a lot, Drew. I've setup the new repository and a service account token in the secrets for this repo. I've also renamed the owner in your PR. I think it's ready to go 🚀

@dustin-decker dustin-decker merged commit d1c7294 into trufflesecurity:main Apr 15, 2022
@drewstinnett
Copy link
Contributor Author

Looks awesome Dustin! Feel free to ping me if anything is wonky and thanks!!

bill-rich pushed a commit that referenced this pull request May 2, 2022
* Initial go at homebrew

* Adding end of file new line

* Readme cleanup

* rename tap owner

Co-authored-by: Dustin Decker <dustin@trufflesec.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants