Marcus,
It can certainly be removed but is there a harm in having it there. As
ext/standard grows there is a good chance it'll be broken into
sections, in which case decencies may become relevant.
On 17-Dec-07, at 8:11 AM, Marcus Boerger wrote:
Hello Ilia,
ZEND_MOD_REQUIRED("standard") ?????
we do not do that for any other ext. Care to explain why it is
needed for exif?
best regards
marcus
Sunday, December 16, 2007, 6:14:12 PM, you wrote:
iliaa Sun Dec 16 17:14:12 2007 UTC
Modified files: (Branch: PHP_5_3)
/php-src/ext/exif exif.c
Log:
Fixed bug #43606 (define missing depencies of the exif extension)
http://cvs.php.net/viewvc.cgi/php-src/ext/exif/exif.c?r1=1.173.2.5.2.20.2.1&r2=1.173.2.5.2.20.2.2&diff_format=u
Index: php-src/ext/exif/exif.c
diff -u php-src/ext/exif/exif.c:1.173.2.5.2.20.2.1
php-src/ext/exif/exif.c:1.173.2.5.2.20.2.2
--- php-src/ext/exif/exif.c:1.173.2.5.2.20.2.1 Thu Sep 27 18:00:38
2007
+++ php-src/ext/exif/exif.c Sun Dec 16 17:14:11 2007
@@ -17,7 +17,7 @@
+
----------------------------------------------------------------------+
*/
-/* $Id: exif.c,v 1.173.2.5.2.20.2.1 2007/09/27 18:00:38 dmitry Exp
$ */
+/* $Id: exif.c,v 1.173.2.5.2.20.2.2 2007/12/16 17:14:11 iliaa Exp
$ */
/* ToDos
*
@@ -142,7 +142,7 @@
};
/* }}} */
-#define EXIF_VERSION "1.4 $Id: exif.c,v 1.173.2.5.2.20.2.1
2007/09/27 18:00:38 dmitry Exp $"
+#define EXIF_VERSION "1.4 $Id: exif.c,v 1.173.2.5.2.20.2.2
2007/12/16 17:14:11 iliaa Exp $"
/* {{{ PHP_MINFO_FUNCTION
*/
@@ -241,12 +241,21 @@
}
/* }}} */
+/* {{{ exif dependencies */
+static const zend_module_dep exif_module_deps[] = {
+ ZEND_MOD_REQUIRED("standard")
+ #if EXIF_USE_MBSTRING
+ ZEND_MOD_REQUIRED("mbstring")
+ #endif
+ {NULL, NULL, NULL}
+};
+/* }}} */
+
/* {{{ exif_module_entry
*/
zend_module_entry exif_module_entry = {
-#if ZEND_MODULE_API_NO >= 20010901
- STANDARD_MODULE_HEADER,
-#endif
+ STANDARD_MODULE_HEADER_EX, NULL,
+ exif_module_deps,
"exif",
exif_functions,
PHP_MINIT(exif),
Best regards,
Marcus
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
Ilia Alshanetsky
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php