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

Build error 1.0.1 #22

Open
wojons opened this issue Jan 16, 2014 · 5 comments
Open

Build error 1.0.1 #22

wojons opened this issue Jan 16, 2014 · 5 comments

Comments

@wojons
Copy link

wojons commented Jan 16, 2014

$ sudo make
/bin/bash /tmp/forp-PHP-profiler-1.0.1/libtool --mode=compile cc  -I. -I/tmp/forp-PHP-profiler-1.0.1 -DPHP_ATOM_INC -I/tmp/forp-PHP-profiler-1.0.1/include -I/tmp/forp-PHP-profiler-1.0.1/main -I/tmp/forp-PHP-profiler-1.0.1 -I/usr/include/php5 -I/usr/include/php5/main -I/usr/include/php5/TSRM -I/usr/include/php5/Zend -I/usr/include/php5/ext -I/usr/include/php5/ext/date/lib  -DHAVE_CONFIG_H  -g -O2   -c /tmp/forp-PHP-profiler-1.0.1/php_forp.c -o php_forp.lo 
libtool: compile:  cc -I. -I/tmp/forp-PHP-profiler-1.0.1 -DPHP_ATOM_INC -I/tmp/forp-PHP-profiler-1.0.1/include -I/tmp/forp-PHP-profiler-1.0.1/main -I/tmp/forp-PHP-profiler-1.0.1 -I/usr/include/php5 -I/usr/include/php5/main -I/usr/include/php5/TSRM -I/usr/include/php5/Zend -I/usr/include/php5/ext -I/usr/include/php5/ext/date/lib -DHAVE_CONFIG_H -g -O2 -c /tmp/forp-PHP-profiler-1.0.1/php_forp.c  -fPIC -DPIC -o .libs/php_forp.o
/tmp/forp-PHP-profiler-1.0.1/php_forp.c: In function 'zm_deactivate_forp':
/tmp/forp-PHP-profiler-1.0.1/php_forp.c:152:26: error: lvalue required as left operand of assignment
/tmp/forp-PHP-profiler-1.0.1/php_forp.c:156:35: warning: assignment from incompatible pointer type [enabled by default]
make: *** [php_forp.lo] Error 1
PHP 5.5.7-1~dotdeb.1 (cli) (built: Dec 14 2013 21:40:10) 
Copyright (c) 1997-2013 The PHP Group
Zend Engine v2.5.0, Copyright (c) 1998-2013 Zend Technologies
    with Zend OPcache v7.0.3-dev, Copyright (c) 1999-2013, by Zend Technologies
@wojons
Copy link
Author

wojons commented Jan 16, 2014

It builds fine on master so it maybe time soon to release 1.0.2

@wojons
Copy link
Author

wojons commented Jan 16, 2014

This may be related to #20

2014/01/15 16:41:50 [error] 1646#0: *93 recv() failed (104: Connection reset by peer) while reading response header from upstream, client: 192.168.0.2, server: example.com, request: "POST /login HTTP/1.1", upstream: "fastcgi://127.0.0.1:9000", host: "192.168.0.3:8080"

@wojons
Copy link
Author

wojons commented Jan 16, 2014

I am seeing the error in fpm i am not sure if the error is in cli i know above i dumped my cli version number since its easy to get.

@wojons
Copy link
Author

wojons commented Jan 16, 2014

Also wanted to add that i have a following version working fine on a different server so it may be a 5.5 issue.

PHP 5.4.21-1~dotdeb.1 (cli) (built: Nov  3 2013 06:56:41) 
Copyright (c) 1997-2013 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2013 Zend Technologies

@crowcrow
Copy link

Hi!
Tried to compile for:

PHP 5.5.12 (cli) (built: Jun 19 2014 19:34:01) 
Copyright (c) 1997-2014 The PHP Group
Zend Engine v2.5.0, Copyright (c) 1998-2014 Zend Technologies
    with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2014, by Zend Technologies

and make failed with Error 1:

/usr/local/src/forp/forp.c: In function ‘forp_open_node’:
/usr/local/src/forp/forp.c:201:13: error: too few arguments to function ‘zend_vm_stack_get_args_count’
In file included from /usr/include/php/Zend/zend_API.h:30:0,
                 from /usr/include/php/main/php.h:38,
                 from /usr/local/src/forp/forp.c:23:
/usr/include/php/Zend/zend_execute.h:328:31: note: declared here
make: *** [forp.lo] Error 1

made a change in forp.c - cb3dfeb

-            if(zend_vm_stack_get_args_count() > 0) {
+            if(zend_vm_stack_get_args_count(TSRMLS_C) > 0) {

and the build finished successfully passing all the tests.

Will continue to monitor for any issues that may arise during the usage.

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

2 participants