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

Fix PECL problem #24913

Merged
merged 1 commit into from Dec 6, 2020
Merged

Fix PECL problem #24913

merged 1 commit into from Dec 6, 2020

Conversation

veblush
Copy link
Contributor

@veblush veblush commented Dec 5, 2020

Currently distribtest.php7_macos_x64_None is failing. This is believed to be caused by Homebrew bug which fails to install pecl correctly on MacOS.

running: make INSTALL_ROOT="/private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev" install
Installing shared extensions:     /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2/7.2.28/pecl/20170718/
running: find "/private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev" | xargs ls -dils
10548148     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev
10555091     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr
10555092     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local
10555093     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar
10555094     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2
10555095     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2/7.2.28
10555096     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2/7.2.28/pecl
10555097     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2/7.2.28/pecl/20170718
10555098 16584 -rwxr-xr-x  1 root  wheel  8488844 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar/php@7.2/7.2.28/pecl/20170718/grpc.so

Build process completed successfully
Installing '/usr/local/Cellar/php@7.2/7.2.28/pecl/20170718/grpc.so'

Warning: mkdir(): File exists in System.php on line 294
PHP Warning:  mkdir(): File exists in /usr/local/Cellar/php@7.2/7.2.28/share/php@7.2/pear/System.php on line 294

Warning: mkdir(): File exists in /usr/local/Cellar/php@7.2/7.2.28/share/php@7.2/pear/System.php on line 294
ERROR: failed to mkdir /usr/local/Cellar/php@7.2/7.2.28/pecl/20170718

Related to Homebrew/homebrew-core#41081

@veblush veblush added lang/php platform/macOS area/build release notes: no Indicates if PR should not be in release notes labels Dec 5, 2020
@veblush
Copy link
Contributor Author

veblush commented Dec 5, 2020

grpc_build_artifacts_multiplatform: passed (build_package.csharp_package_unity_linux failed but it's not relevant)

@veblush veblush marked this pull request as ready for review December 5, 2020 18:22
Copy link
Contributor

@stanley-cheung stanley-cheung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@jtattermusch
Copy link
Contributor

Merging to fix the breakage.

@jtattermusch jtattermusch merged commit f695914 into grpc:master Dec 6, 2020
yulin-liang added a commit to yulin-liang/grpc that referenced this pull request Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build lang/php platform/macOS release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants