Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#709 closed defect (fixed)

Geos-php extenstion, tmp undeclared while compiling

Reported by: KayMadejski Owned by: strk
Priority: minor Milestone: 3.6.1
Component: PHP Version: main
Severity: Significant Keywords:
Cc:

Description

While compiling latest trunk:

php/geos.c: In function 'Gen_create_obj':
php/geos.c:174:50: error: 'tmp' undeclared (first use in this function)

I've just changed "(void*) &tmp" to "NULL" and it compiled, but I'm in no position to say if it's ok.

I've found possible relevant info: http://stackoverflow.com/questions/14105529/writing-a-c-extension-for-php-5-4-example-code-is-obsolete

Change History (5)

comment:1 by strk, 9 years ago

Status: newassigned

What version of PHP are you building against ? Did "make check" run under php dir work fine ? If so, I'll commit :)

comment:2 by strk, 9 years ago

I've just tried with php-5.3.2 and I get this error on "make check":

Fatal error: Call to undefined function hex2bin() in /home/strk/src/geos/libgeos-trunk/php/test/test.php on line 2193

comment:3 by strk, 9 years ago

It looks like hex2bin requires 5.4.0+, so that's an unrelated issue (but worth fixing) http://php.net/manual/it/function.hex2bin.php

comment:4 by strk, 9 years ago

Resolution: fixed
Status: assignedclosed

Added replacement, tests pass now with 5.3.2 too. Committed as r4027 in trunk (3.5.0) and r4028 in 3.4 branch (3.4.3)

comment:5 by strk, 8 years ago

Milestone: 3.4.33.6.1

Ticket retargeted after milestone deleted

Note: See TracTickets for help on using tickets.