Skip to content

Conversation

@soilSpoon
Copy link
Contributor

brew list does not support TTY. So a raises a warning or error when used with pipe.
This PR solves the above situation.

Ex)

❯ /opt/homebrew/bin/brew list | grep php
Error: Calling `brew list` to only list formulae is disabled! Use `brew list --formula` instead.
❯ /usr/local/bin/brew list | grep go
Warning: Calling `brew list` to only list formulae is deprecated! Use `brew list --formula` instead.
go

Reference

Homebrew/brew#8851
Homebrew/brew#8229

@drbyte
Copy link
Contributor

drbyte commented Nov 27, 2020

Thanks for catching this.

We should probably release this fix quickly as it may be affecting some installs, as mentioned in the linked Valet issue.

@mattstauffer mattstauffer merged commit 3f0f157 into laravel:master Nov 27, 2020
@soilSpoon soilSpoon deleted the feature/brew branch November 28, 2020 07:18
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.

3 participants