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

Upgrading 7.4.18 with patch included #307

Closed
scottperlitch opened this issue May 5, 2021 · 1 comment
Closed

Upgrading 7.4.18 with patch included #307

scottperlitch opened this issue May 5, 2021 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@scottperlitch
Copy link

Describe the bug

My hypothesis is that brew upgrade shivammathur/php/php@7.4 is not installing the latest patch (fc256da)

Running brew upgrade shivammathur/php/php@7.4 only outputs: "Warning: shivammathur/php/php@7.4 7.4.18 already installed". Is this the expected behavior?

Is there a way to verify if the patch has been applied using phpinfo()?

PHP versions

7.4.18.

To Reproduce

brew install php@7.4
brew upgrade shivammathur/php/php@7.4

Expected behavior

The patch fixes a postgres / pdo bug. I am still seeing an error related to this bug when using Laravel Valet for development.

Screenshots/Logs

Additional context

The PHP bug that was patched: php/php-src#6801

Are you willing to submit a PR?

Yes

@scottperlitch scottperlitch added the bug Something isn't working label May 5, 2021
@shivammathur
Copy link
Owner

@scottperlitch

Reinstall using the formula from the tap. That should fix it.

brew update
brew reinstall shivammathur/php/php@7.4
brew link --force --overwrite php@7.4

@shivammathur shivammathur added question Further information is requested and removed bug Something isn't working labels Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants