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

Fail to make it #5

Closed
hyper0x opened this issue Jun 1, 2016 · 11 comments
Closed

Fail to make it #5

hyper0x opened this issue Jun 1, 2016 · 11 comments

Comments

@hyper0x
Copy link
Contributor

hyper0x commented Jun 1, 2016

Hi,

I ran make under the project's dir, and got:

$ make
go install ./zend
# github.com/hyper-carrot/php-go/zend
In file included from zend/szend.c:10:
In file included from /usr/local/Cellar/php56/5.6.22/include/php/Zend/zend_API.h:26:
In file included from /usr/local/Cellar/php56/5.6.22/include/php/Zend/zend_modules.h:25:
In file included from /usr/local/Cellar/php56/5.6.22/include/php/Zend/zend.h:51:
In file included from /usr/local/Cellar/php56/5.6.22/include/php/Zend/zend_config.h:1:
In file included from /usr/local/Cellar/php56/5.6.22/include/php/main/../main/php_config.h:2530:
/usr/include/sys/types.h:92:23: error: cannot combine with previous 'int' declaration specifier
zend/sztypes.h:5:23: note: expanded from macro 'uint'
make: *** [all] Error 2

I don't know what is the matter. My Env:

OS: OS X 10.11.5
Go version: 1.6.1
PHP version: 5.6.22 (install with homebrew)
@kitech
Copy link
Owner

kitech commented Jun 1, 2016

When I try little change, that's compile error can be fixed.
But now golang(<=1.6) for OS X not support -linkshared parameter,
so one can only use gozend, can not use phpgo.

@hyper0x
Copy link
Contributor Author

hyper0x commented Jun 2, 2016

OK, looking forward to improving. Would you provide the point in time?

@kitech
Copy link
Owner

kitech commented Jun 2, 2016

Of couse.
I can compile zend and phpgo packages to .a on OS X. Next step don't known how link it.
I can just fix the first half of this bug now.

@kitech
Copy link
Owner

kitech commented Jun 2, 2016

Another, I try the brew install php56, this package not contains embed library named like libphp5.so.
Maybe need a custom build php package for full feature.
Much work to do about OS X platform.

@hyper0x
Copy link
Contributor Author

hyper0x commented Jun 2, 2016

@kitech OK, Thank you.

@joostshao
Copy link

两个中国人...

@kitech
Copy link
Owner

kitech commented Jun 3, 2016

@hyper-carrot Thank you too.

@kitech
Copy link
Owner

kitech commented Jun 3, 2016

好棒,只是输出调试信息太多了

@hyper0x
Copy link
Contributor Author

hyper0x commented Jun 6, 2016

@kitech @joostshao English discussions conducive to international collaboration. It's a lesson for open source project on Github.

@kitech
Copy link
Owner

kitech commented Feb 19, 2017

since go 1.8 has full support buildmode=c-shared for MacOS,

with some test and looks fine.

think as fixed if no more report.

@septianw
Copy link

It's been years ago, please, somebody close this issue.

@hyper0x hyper0x closed this as completed Mar 23, 2019
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

No branches or pull requests

4 participants