Re: cvs: php4 / configure.in

The comment is bogus the way it is written, because it's lacking the security implications of turning register_globals on. Whenever we encourage people to turn it on, for whatever reason, we should also point them to the URL that covers the implications. I think that a comment does belong there, but it should be clarified. I volunteer to clarify it... Zeev At 14:27 02/05/2002, Thies C. Arntzen wrote:

On Thu, May 02, 2002 at 08:32:02AM -0000, Jani Taskinen wrote:

sniper                Thu May  2 04:32:02 2002 EDT

Modified files:

    /php4     configure.in

Log: Remove bogus warning.

    i really don't think this warning is bogus. why remove it,
    sascha added it for a reason.

    tc

Index: php4/configure.in diff -u php4/configure.in:1.341 php4/configure.in:1.342 --- php4/configure.in:1.341 Wed May 1 09:34:05 2002 +++ php4/configure.in Thu May 2 04:31:55 2002 @@ -1,4 +1,4 @@ -dnl ## $Id: configure.in,v 1.341 2002/05/01 13:34:05 sas Exp $ -*- sh -*- +dnl ## $Id: configure.in,v 1.342 2002/05/02 08:31:55 sniper Exp $ -*- sh -*- dnl ## Process this file with autoconf to produce a configure script. divert(1) @@ -1287,11 +1287,6 @@

 | process, you are bound by the terms of this license agreement.     |

| If you do not agree with the terms of this license, you must abort |

 | the installation process at this point.                            |

-+--------------------------------------------------------------------+

-|                              *** WARNING ***                       |
-|                                                                    |

-| Remember to enable register_globals in your PHP.INI file, if your |

-| software depends on it.                                            |

+--------------------------------------------------------------------+ -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php