Skip to content

Commit

Permalink
Merge pull request #2222 from stanley-cheung/fix_php_extension_osx_co…
Browse files Browse the repository at this point in the history
…nfigure

php extension: do not link rt in osx
  • Loading branch information
murgatroid99 committed Jun 25, 2015
2 parents 88f8e94 + 76ed0cc commit d42ad96
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 7 deletions.
23 changes: 18 additions & 5 deletions src/php/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,27 @@ This directory contains source code for PHP implementation of gRPC layered on sh

#Status

Pre-Alpha : This gRPC PHP implementation is work-in-progress and is not expected to work yet.
Alpha : Ready for early adopters

## ENVIRONMENT

Prerequisite: PHP 5.5 or later, PHPUnit, pecl
Prerequisite: PHP 5.5 or later, `phpunit`, `pecl`

Linux:

```sh
$ sudo apt-get install php5 php5-dev phpunit php-pear
```

OS X:

```sh
sudo apt-get install php5 php5-dev phpunit php-pear
$ curl https://phar.phpunit.de/phpunit.phar -o phpunit.phar
$ chmod +x phpunit.phar
$ sudo mv phpunit.phar /usr/local/bin/phpunit

$ curl -O http://pear.php.net/go-pear.phar
$ sudo php -d detect_unicode=0 go-pear.phar
```

## Build from Homebrew
Expand Down Expand Up @@ -71,8 +84,8 @@ $ make
$ sudo make install
```

In your php.ini file, add the line `extension=grpc.so` to load the extension
at PHP startup.
(Optional) In your php.ini file, add the line `extension=grpc.so` to load
the extension at PHP startup.

Install Composer

Expand Down
9 changes: 7 additions & 2 deletions src/php/ext/grpc/config.m4
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,13 @@ if test "$PHP_GRPC" != "no"; then
PHP_ADD_LIBRARY(dl,,GRPC_SHARED_LIBADD)
PHP_ADD_LIBRARY(dl)

PHP_ADD_LIBRARY(rt,,GRPC_SHARED_LIBADD)
PHP_ADD_LIBRARY(rt)
case $host in
*darwin*) ;;
*)
PHP_ADD_LIBRARY(rt,,GRPC_SHARED_LIBADD)
PHP_ADD_LIBRARY(rt)
;;
esac

GRPC_LIBDIR=$GRPC_DIR/${GRPC_LIB_SUBDIR-lib}

Expand Down

0 comments on commit d42ad96

Please sign in to comment.