|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #59634 fix build #1
Submitted: 2011-02-21 12:17 UTC Modified: 2011-02-22 11:02 UTC
From: Assigned: bd808 (profile)
Status: Closed Package: yaml (PECL)
PHP Version: 5_3 SVN-2011-02-21 (dev) OS: windows
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
Solve the problem:
21 + 24 = ?
Subscribe to this entry?

 [2011-02-21 12:17 UTC]

Here is a 1st patch to fix yaml build on windows:

There are other linker issues but will fix them later.



Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2011-02-22 00:38 UTC]
I applied your changes to config.w32 and parse.c. The change you proposed for yaml.c breaks runtime usage and the change to emit.c is redundant since yaml.h is included in the php_yaml.h header.
 [2011-02-22 05:17 UTC] pierre dot php at gmail dot com
This double declaration in yaml.c is wrong. Do you test the 
build under thread safe mode or normal (nts) mode?
 [2011-02-22 06:04 UTC] pierre dot php at gmail dot com
Applied the required changes directly to svn, hope you don't 
mind. The two last problems were:

- declaration of the yaml globals using PHP_API in yaml.c but 
not in php_yaml.h
- missing flag for static libyaml usage
 [2011-02-22 11:02 UTC]
No worries about the direct commit. The whole point of hosting the project with is to get assistance from others in keeping the code clean and functional.

I have been testing under nts against Ubuntu 10.04's php distribution. Does pecl have a build farm somewhere that would let me test against a wider variety of php distributions and configurations?
PHP Copyright © 2001-2022 The PHP Group
All rights reserved.
Last updated: Tue Jan 18 19:03:14 2022 UTC