Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:20473
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Date: Sat, 26 Nov 2005 03:22:36 +0200 (EET)
Reply-To: Jani Taskinen <sniper@iki.fi>
To: Rasmus Lerdorf <rasmus@lerdorf.com>
cc: internals@lists.php.net
In-Reply-To: <4386C97D.3030402@lerdorf.com>
Message-ID: <Pine.LNX.4.61.0511260318110.30988@arfg.argcubovn.sv>
References: <4386C97D.3030402@lerdorf.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
Subject: Re: [PHP-DEV] Fixing this date mess
From: sniper@iki.fi (Jani Taskinen)

On Fri, 25 Nov 2005, Rasmus Lerdorf wrote:

>
> We need to fix this date thing along with the Digest Auth mixup and get a 
> 5.1.1 out the door soon.  I see Ilia committed a fix for the Digest Auth 
> thing, so assuming that this fix works, we just need to figure out what to do 
> with this date class.  We can either roll back:
>
> http://cvs.php.net/diff.php/php-src/ext/date/php_date.c?r1=1.43.2.20&r2=1.43.2.21&ty=u
>
> or perhaps a better option is to rename that internal class for now until we

     Either keep it as is or nuke it forever. But please, no pingpong here.

> have a better idea of what this class is going to look like and how we 
> transition pear/Date users.  Naming it date_ex (for experimental) or something 
> like that for now lets us keep the class constants as they are and hopefully 
> Derick and Pierre can come to some sort of truce on the actual implementation.

     I wouldn't hold my breath waiting for that to happen.

> I don't think having this mostly empty date class placeholder helps anybody 
> right now.

     It already helped. People woke up. :) Make a big fat notice about it and
     let it blow things up and we end up with something better.

     --Jani