Re: Re: cvs: php-src(PHP_5_2) / NEWS /main rfc1867.crfc1867.h

From: Date: Sun, 23 Jul 2006 15:51:07 +0000
Subject: Re: Re: cvs: php-src(PHP_5_2) / NEWS /main rfc1867.crfc1867.h
References: 1 2 3 4 5 6 7 8 9 10 11  Groups: php.internals 
Request: Send a blank email to internals+get-24909@lists.php.net to get a copy of this message
Hello Stefan, Sunday, July 23, 2006, 3:39:35 AM, you wrote: >> If you'd be grown up yourself, you would just commit to HEAD instead >> of spending time on meaningless insults. > Commiting to HEAD is not possible. How often do I have to tell that. > Because of the unicode changes the patch has to be majorly rewritten. > Additionally it is simply impossible to commit to HEAD and test against > it, because the fileupload code in CVS HEAD is broken because of all the > unicode changes. Nonetheless you are inventing a new feature here. One that has been asked for and for which patches have often been offered in the past. The general idea was to add the feature once we have a working patch, so in this sense it was ok to add the patch. However i am not sure you didn't try to sneak the stuff in as Pierre would have called it, since i doubt it was actively on any todo or any agreement for 5.2. > There is no point in rewritting the patch while the fileupload code in > CVS HEAD simply segfaults if you upload a file. Then first of all we should fix that part and not invent new features. [...] Best regards, Marcus

« previous php.internals (#24909) next »