Bug 511018 - syck-php doesn't build with PHP 5.3.0
Summary: syck-php doesn't build with PHP 5.3.0
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: syck
Version: 12
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Oliver Falk
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-07-13 09:40 UTC by Remi Collet
Modified: 2010-02-17 15:45 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-02-17 15:45:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Remi Collet 2009-07-13 09:40:34 UTC
I'm working on rebuilding all PHP extension against new 5.3.0 version.

syck fails :
/home/extras/BUILD/syck-0.61+svn231+patches/ext/php/phpext.c:24: error: static declaration of 'nan' follows non-static declaration
/home/extras/BUILD/syck-0.61+svn231+patches/ext/php/phpext.c: In function 'nan':
/home/extras/BUILD/syck-0.61+svn231+patches/ext/php/phpext.c:24: error: number of arguments doesn't match prototype
/usr/include/bits/mathcalls.h:225: error: prototype declaration
/home/extras/BUILD/syck-0.61+svn231+patches/ext/php/phpext.c: In function 'php_syck_handler':
/home/extras/BUILD/syck-0.61+svn231+patches/ext/php/phpext.c:223: error: too few arguments to function 'nan'
...

Upstream seems not active for syck ?

Need to be investigated.

Does http://pecl.php.net/package/syck is the same extension ?

Comment 1 Remi Collet 2009-07-14 06:34:17 UTC
Seems the PECL extension is the same but maintained one.

The build issue (nan function) seems not PHP 5.3 related...

I've add a little patch which solves this (same change found in http://github.com/why/syck/blob/a4f241be5d247853aea6127d02dbdedd8a1dd477/ext/php/phpext.c)

syck-0.61-8.3 build in Rawhide.

PHP extension build, install and load fine.

Please, check my changes and close this bug if all is ok.

Comment 2 Bug Zapper 2009-11-16 10:47:40 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 12 development cycle.
Changing version to '12'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 3 Oliver Falk 2010-02-17 15:45:33 UTC
I missed this one... :-/
Everything works. However, the package could need more love. Especially because there's a new upstream version hosted somewhere else and all the extensions are moved to other locations...
I hope I can do this at some point in time.


Note You need to log in before you can comment on or make changes to this bug.