Working with Rasmus on PHP bug fixes
--
PHP Development Mailing List http://www.php.net/
To unsubscribe, visit: http://www.php.net/unsub.php
Try this:
/*** array your_ext_func() ***/
ZEND_FUNCTION(your_ext_func)
{
zval *val_ret, *val_str;
char *output;
int ret;
output = (char *)emalloc( 100 );
some code to set output and ret
array_init(return_value);
MAKE_STD_ZVAL(val_ret);
I have seen the configure error before. Don't know if this is what
you are doing, but for me it was using phpize from 4.2.x and trying
to build 4.3 extensions.
Hope this helps,
Brian
At 11:27 AM -0700 9/12/02, Brian Lalor wrote:
I just grabbed PHP from CVS and tried to go through the
Has anybody looked into doing a include directive for the php.ini file?
Something like what Apache has?
http://httpd.apache.org/docs/mod/core.html#include
Thoughts?
Cheers,
Brian
--
PHP Development Mailing List http://www.php.net/
To unsubscribe, visit: http://www.php.net/unsub.php
or whatever).
Anyway, I don't know. I give this maybe +.25 for right now, see what
reactions are.
I could put together a quick patch for this if people thought it would be
useful. If you guys like the idea, lemme know, and I'll patch it.
Devon
Original Message:
-
From: Brian France
I have created a patch for two new .ini directives:
include_ini = filename ; relative or full path
include_ini_dir = full directory path
The patch is only in php_ini.c and does the following:
Added 2 new static zend_llist: include_ini_list include_ini_dir_list
Add a case for include_ini and
At 11:25 AM +0900 9/12/02, Yasuo Ohgaki wrote:
How this could be useful?
I work for a company that is moving towards PHP. We have our own
package system for installing software, something like FreeBSD ports.
I am the one that builds the PHP dso package. The PHP package
could/will be
Sorry that should have been:
-
[PHP]
include_ini_dir = include
-
Brian
--
PHP Development Mailing List http://www.php.net/
To unsubscribe, visit: http://www.php.net/unsub.php
At 3:09 AM +0100 9/12/02, Wez Furlong wrote:
Arrgh! You should *really* be working against HEAD because there have
been so many changes since we branched 4.2 - and some of those were
related to php.ini detection.
It's a LOT easier to deal with patches against HEAD (take a look in
This is just a ping test to see if my co-location company got off of
the spews.org black list.
Brian
--
PHP Development Mailing List http://www.php.net/
To unsubscribe, visit: http://www.php.net/unsub.php
Personally I don't care what is the default extensions are as long as
I can change them.
(I have written this email several times, but never sent it or I may
have, but don't remember if so sorry if repeating).
Here are my problems with extensions:
A) Extensions that change the core code when
At 10:49 AM +0100 9/1/02, James Cox wrote:
I vote to remove mbstring as a default module.
I completely agree!
I do have one request, when the time comes to remove it from the
default also change the PHP core code so that it can be built as a
shared extension and loaded on the fly. Currently
At 6:29 PM +0100 9/1/02, James Cox wrote:
Where is your patch?
The patch basically renames php_treat_data to php_treat_data_default,
creates a function pointer called php_treat_data that is defaulted to
php_treat_data_default, removes all mbstrings references in php_main.h
and makes mbstring.c
At 8:47 PM +0200 9/1/02, Marcus Börger wrote:
At 20:38 01.09.2002, Brian France wrote:
At 6:29 PM +0100 9/1/02, James Cox wrote:
Where is your patch?
The patch basically renames php_treat_data to php_treat_data_default,
creates a function pointer called php_treat_data that is defaulted
I know this is ugly, but what about making the extensions handle it themselves?
Your example of session_pgsql:
In the extension init code:
ext_enabled = 1;
if ( dlsym( NULL, psql_module_entry ) == NULL
dlsym( NULL, _psql_module_entry ) == NULL )
{
// print some warning about
At 9:25 AM +0900 9/2/02, Yasuo Ohgaki wrote:
As all of us know, PHP crashes easily when module is loaded
improperly.
Has anybody tracked down why this is?
This could have been the problem, but is fixed now:
http://bugs.php.net/?id=17643
(I think this link is right, bugs is down right now)
I tried to build xmlrpc as a shared extension (phpize, ./configure,
make), but when it builds it didn't have dependences for libexpat and
libiconv, but the expat, libiconv and XMLRPC_ functions are undefined.
While trying to fix this I rewrote the config.m4 to handle all the
following build
At 9:00 PM +0200 7/17/02, [EMAIL PROTECTED] wrote:
On Wed, 17 Jul 2002, Jani Taskinen wrote:
Can't they be merged and use #ifdef's to enable
the multibyte support?
Personally I don't like the idea of having a module change the PHP
core code with a #ifdef like mbstring does now. I
Hello,
I have a url that ends with ?.value=1, but PHP returns this in
$_REQUEST as _value instead of .value. I believe this is because of
this code:
main/php_variables.c
104 /* ensure that we don't have spaces or dots in the
variable name (not binary safe) */
105 for
Great!
Once the patch is checked in feel free to close the bug report or let
me know and I will close it.
Thanks!
Brian
At 9:59 PM +0300 6/26/02, Jani Taskinen wrote:
On Wed, 26 Jun 2002, Brian France wrote:
Does this fix the problem with configure?
http://bugs.php.net/bug.php?id=17977
I have been trying to build session support as a shared extension and
here is what I have found. It works fine if you us the phpize,
./configure, make method, but if you try building it shared while
building PHP it will not work.
./configure --with-layout=GNU --enable-session=shared
too.
If you do decouple mbstring, please make sure that the mbstring headers
are still installed in the pear include directory too!
--Wez.
On 20/06/02, Brian France [EMAIL PROTECTED] wrote:
Ok, what does exif need from mbstring, code wise? Taking a look at
it I don't see any references
At 6:57 PM +0100 6/21/02, Wez Furlong wrote:
If you do decouple mbstring, please make sure that the mbstring headers
are still installed in the pear include directory too!
If mbstring is built as a shared extension, you need to ensure that it's
headers are also installed in the pear header
At 7:23 PM +0200 6/21/02, Marcus Börger wrote:
What i wanted to stree out is that exif uses functions from mbstring
if present.
That means if mbstring is not present it does not use these and the
user has to do that stuff for his own.
If mbstring is compiled as a shared extension both exif and
At 9:56 PM +0200 6/21/02, Marcus Börger wrote:
At 20:29 21.06.2002, Brian France wrote:
At 7:23 PM +0200 6/21/02, Marcus Börger wrote:
What i wanted to stree out is that exif uses functions from
mbstring if present.
That means if mbstring is not present it does not use these and
the user has
Ok, what does exif need from mbstring, code wise? Taking a look at
it I don't see any references to mbstring in the code.
Thanks,
Brian
At 8:27 PM +0200 6/20/02, Marcus Börger wrote:
You will break ext/exif when removing mbstrings current integration
At 04:17 20.06.2002, you wrote:
I am
, but it is
rather small compared to the one if you build it with phpize,
./configure, make in the ext/mysql directory.
Thanks,
Brian
At 8:44 AM +0900 6/21/02, Yasuo Ohgaki wrote:
Brian France wrote:
Ok, what does exif need from mbstring, code wise? Taking a look at
it I don't see any references
At 2:57 AM +0300 6/8/02, Zeev Suraski wrote:
We're on the job. I generally think you're right, we have to do
some more thinking but chances are we will change the shutdown order
to be reversed.
Just a FYI, may want to do the same for zend_extension (reverse
order). I think that is a
Zend Engine unloading extension in the wrong order! (Re-post due to
no response)
I have been tracking down a core dump with my php extension when it
is unloading some of its shared libraries. I have shared extension
that is link against libexpat and will load fine if it is the only on
in
--- Brian France [EMAIL PROTECTED] wrote:
Zend Engine unloading extension in the wrong order! (Re-post due to
no response)
I have been tracking down a core dump with my php extension when it
is unloading some of its shared libraries. I have shared extension
that is link against libexpat
At 10:53 PM +0300 6/7/02, Andi Gutmans wrote:
Can you check and see if using zend_hash_graceful_reverse_destroy()
fixes your problem?
Andi
At 11:44 AM 6/7/2002 -0700, Brian France wrote:
Zend Engine unloading extension in the wrong order! (Re-post due to
no response)
I have been tracking down
Somebody from PRESENCE-GROUP.COM is subscribed to php-dev list and is
filtering mail through spamcop or something program. I am getting
them to.
Brian
At 10:58 PM +0300 6/7/02, Andi Gutmans wrote:
Any idea why I got this when posting to php-dev?
Andi
Delivered-To: [EMAIL PROTECTED]
Sender:
Yes, zend_hash_graceful_reverse_destroy() fixes the problem as well.
Is this a better solution for the fix?
Thanks,
Brian
At 11:06 PM +0300 6/7/02, Andi Gutmans wrote:
At 01:00 PM 6/7/2002 -0700, Brian France wrote:
I will try that, but I was worried that the pDestructor function
would
I have been tracking down a core dump with my php extension when it
is unloading some of its shared libraries. I have shared extension
that is link against libexpat and will load fine if it is the only on
in the php.ini. If I then load xml.so extension then my extension it
works fine, but
I am changing the way I build my PHP extension and am running into
some problems. I was originally would build it by copying the
directory foo_bar into the ext directory and added
--enable-foo_bar=shared to the configure line of PHP and build PHP.
Everything worked great! Now I am looking to
, Whether to
build $1 as dynamic module)
Should I just submit a bug report on this or am I still missing something?
Thanks,
Brian
At 1:33 PM -0700 5/24/02, Brian France wrote:
I am changing the way I build my PHP extension and am running into
some problems. I was originally would build
)
To:
AC_DEFINE_UNQUOTED([COMPILE_DL_]translit($1,a-z_-,A-Z__), 1, Whether
to build $1 as dynamic module)
Should I just submit a bug report on this or am I still missing something?
Thanks,
Brian
At 1:33 PM -0700 5/24/02, Brian France wrote:
I am changing the way I build my PHP extension and am
37 matches
Mail list logo