Gwynne wrote:
On Jul 3, 2006, at 7:57 PM, Rasmus Lerdorf wrote:
The general consensus seems to be that I should show as much info as is reasonably possible. For this, extensions need to be able to register the summary information they can provide. I've come up with a couple of different ideas for the syntax on this:
1) Add a new parameter to PHP_NEW_EXTENSION(), i.e.:
PHP_NEW_EXTENSION(standard, array.c base64.c basic_functions.c browscap.c crc32.c # \ etc,
                       ,,,, \
                       [Standard functions
Regex type:--with-regex=${REGEX_TYPE}])
2) AC_DEFUN() a new macro, i.e.:
    PHP_SUMMARY_INFO([Standard functions], [Extension: Enabled
Regex type: --with-regex=${REGEX_TYPE}])
These are just random examples based on the standard extension, obviously. Anyone like either of these or have a better idea?
I really don't think it is worth changing/adding to the extension API for this in any way.

If we can't do a simple summary with what we already have, let's just skip it. And as far as I am concerned, showing as much as possible isn't the goal here. The goal for me would be to get a summary that fits on a single 80x25 page without scrolling. If it is so long that it scrolls off, then we haven't gained much. It should be a quick-summary that you can glance at and quickly pick out if everything looks ok.

That was my original intention, but I was shouted down twice already when I suggested it. I'm willing to implement both options, maybe put the one that changes the extension API in HEAD only?

I still don't think we need to make the extension API any more complex. We already have an extension summary in the config.m4 file and it should be possible to fiddle with the autoconf macros to pull it from there. Forcing people to write this stuff twice doesn't make much sense.

Go with your initial instict and do a nice short and concise summary. I see no reason for a phpinfo() like thing since you can just type sapi/cli/php -i to get that.

-Rasmus

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to