php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #37162 WDDX module doesn't build non-bundled
Submitted: 2006-04-22 10:39 UTC Modified: 2006-04-23 16:02 UTC
From: jdolecek at NetBSD dot org Assigned:
Status: Closed Package: WDDX related
PHP Version: 5.1.2 OS: NetBSD
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: jdolecek at NetBSD dot org
New email:
PHP Version: OS:

 

 [2006-04-22 10:39 UTC] jdolecek at NetBSD dot org 
Description:
------------
When WDDX module is compiled separately from main PHP build, the module contents are not actually compiled due to #ifdef HAVE_WDDX. Since config.h is not included,HAVE_WDDX is not defined in this case and empty .o file is produced.

This problem also affects PHP 4.4.2.

Fix is to include config.h on top:


--- ext/wddx/wddx.c.orig        2006-04-22 12:18:32.000000000 +0200
+++ ext/wddx/wddx.c
@@ -20,2 +20,6 @@
 
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "php.h"


Reproduce code:
---------------
Build as separate module (i.e. not compiled into php) and load via extension=wddx.so

Expected result:
----------------
Module loads

Actual result:
--------------
php prints warning when loading the extension:

PHP Warning:  Unknown(): Invalid library (maybe not a PHP library) 'wddx.so'  in Unknown on line 0

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2006-04-23 16:02 UTC] iliaa@php.net 
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Nov 21 12:01:29 2024 UTC