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.

Sign up for GitHub

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

Jump to bottom

Documentation bug: brew bottle --merge help says it take a formula, actually takes a JSON file #4519

Closed
fredemmott opened this issue Jul 20, 2018 · 1 comment
Closed

Documentation bug: brew bottle --merge help says it take a formula, actually takes a JSON file #4519

fredemmott opened this issue Jul 20, 2018 · 1 comment
Assignees
@MikeMcQuaid
Labels
outdated PR was locked due to age

Comments

@fredemmott
Copy link
Contributor 

brew bottle --merge [--keep-old] [--write [--no-commit]] formulae:
    Generate a bottle from a formula and print the new DSL merged into the
    existing formula.

    If --write is passed, write the changes to the formula file. A new
    commit will then be generated unless --no-commit is passed.

It doesn't take formulae - it takes a JS file that's the result of brew bottle [options] --json formula (--json also being undocumented)
@MikeMcQuaid
Copy link
Member

@fredemmott I'll take a look at this. thanks. As a general comment this stuff is a bit of an implementation detail; I'd strongly advise using brew test-bot to build your bottles and we'll accept PRs to help make that more suitable.

@MikeMcQuaid MikeMcQuaid mentioned this issue Jul 23, 2018
Merged
6 tasks
@ghost ghost assigned MikeMcQuaid Jul 23, 2018
@ghost ghost added the in progress Maintainers are working on this label Jul 23, 2018
@ghost ghost removed the in progress Maintainers are working on this label Jul 23, 2018
@lock lock bot added the outdated PR was locked due to age label Aug 22, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MikeMcQuaid MikeMcQuaid

Labels
outdated PR was locked due to age
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MikeMcQuaid @fredemmott