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

ripgrep 0.2.1 (new formula) #5268

Closed
wants to merge 1 commit into from

Conversation

moshen
Copy link
Contributor

@moshen moshen commented Sep 26, 2016

  • Have you followed the guidelines in our Contributing document?

  • Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?

  • Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?

  • Does your submission pass brew audit --new-formula <formula> (after doing brew install <formula>)?

    I can't because of "Error: type mismatch: String given" on brew audit --strict --online brew#1140

    Switched to master and was able to run audit.


@zmwangx zmwangx added the new formula PR adds a new formula to Homebrew/homebrew-core label Sep 26, 2016
@moshen
Copy link
Contributor Author

moshen commented Sep 27, 2016

😦 1 failing check, but when I click the link there are no failing checks.

@bfontaine
Copy link
Contributor

It failed on Sierra:

==> brew install --only-dependencies --build-bottle --verbose ripgrep
Build timed out (after 60 minutes). Marking the build as aborted.

@justinmayer
Copy link
Contributor

@moshen: When you update this PR to pass tests on Sierra, you might as well update it to the just-released ripgrep 0.2.1 version as well. (^_^)

@moshen moshen force-pushed the new-formula-ripgrep branch from 5fdc9f5 to ab43c5d Compare September 27, 2016 17:09
@moshen moshen changed the title ripgrep 0.2.0 (new formula) ripgrep 0.2.1 (new formula) Sep 27, 2016
@moshen
Copy link
Contributor Author

moshen commented Sep 28, 2016

==> brew install --only-dependencies --build-bottle --verbose ripgrep
Build timed out (after 60 minutes). Marking the build as aborted.

Is this related to: #4841 ? Specifically rust being in that list?

@halilim
Copy link
Contributor

halilim commented Sep 30, 2016

BTW, there is a formula in the ripgrep repo itself which uses pre-built binaries.

@wkral
Copy link

wkral commented Sep 30, 2016

Looks like the rust issue was resolved for sierra in #4960 so the build may work now.

@dunn dunn added the CI-requeued PR has been re-added to the queue label Sep 30, 2016
@MikeMcQuaid
Copy link
Member

Thanks again for your contribution to Homebrew! Without people like you submitting PRs we couldn't run this project. You rock!

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI-requeued PR has been re-added to the queue new formula PR adds a new formula to Homebrew/homebrew-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants