Re: cvs: pecl /http KnownIssues.txt config.m4 http_date_api.c http_functions.c http_headers_api.c http_message_api.c http_message_object.c http_request_api.c http_request_object.c http_request_pool_api.c http_requestpool_object.c http_send_api.c http_url_api.c php_http_std_defs.h

Hello Michael,

Sunday, July 24, 2005, 12:30:45 PM, you wrote:

> mike            Sun Jul 24 06:30:45 2005 EDT

>   Modified files:              
>     /pecl/http  KnownIssues.txt config.m4 http_date_api.c 
>                 http_functions.c http_headers_api.c http_message_api.c 
>                 http_message_object.c http_request_api.c 
>                 http_request_object.c http_request_pool_api.c 
>                 http_requestpool_object.c http_send_api.c http_url_api.c 
>                 php_http_std_defs.h 
>   Log:
>   - fix some gcc warnings

[...]

Index: pecl/http/config.m4
diff -u pecl/http/config.m4:1.20 pecl/http/config.m4:1.21
--- pecl/http/config.m4:1.20    Sat Jul 23 17:48:45 2005
+++ pecl/http/config.m4 Sun Jul 24 06:30:45 2005
@@ -1,5 +1,5 @@
 dnl config.m4 for pecl/http
-dnl $Id: config.m4,v 1.20 2005/07/23 21:48:45 helly Exp $
+dnl $Id: config.m4,v 1.21 2005/07/24 10:30:45 mike Exp $
 
 PHP_ARG_ENABLE([http], [whether to enable extended HTTP support],
 [  --enable-http           Enable extended HTTP support])
@@ -79,6 +79,10 @@
        PHP_NEW_EXTENSION([http], $PHP_HTTP_SOURCES, [$ext_shared])
        PHP_SUBST([HTTP_SHARED_LIBADD])
        PHP_ADD_MAKEFILE_FRAGMENT
+       if ! test -z "$HTTP_ODD_WARNINGS"; then
+               CFLAGS=" -g -O2 -W -Wchar-subscripts -Wformat=2 -Wno-format-y2k 
-Wimplicit -Wmissing-braces
-Wunused-variable -Wbad-function-cast -Wpointer-arith -Wsign-compare -Winline"
+               PHP_SUBST([CFLAGS])
+       fi
        AC_DEFINE([HAVE_HTTP], [1], [Have extended HTTP support])
 fi


 That doesn't really help, since i still have them for the complete build.
Thus i cannot build php anymore. There must be a way that works better.
You somehow need to be able to do this for your dev builds alone not
affecting any non ext/http build/developer. The simplest way is to use a
build script for the extension that sets CFLAGS before building it.

regards
marcus