Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:24392
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
DomainKey-Status: good
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=googlemail.com;
        h=received:message-id:date:from:reply-to:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=Cdx5ugPSBlcX2ycRdvsG4Lbr3Uyag49hdpvAN5xEZK3/U3rKKKvDZZ/Yx/Pcuv9wD4M0E3x8V5QnHwkdkIujkP/iqFXoFDVuZIFQFA7DbFInNZhQnwJv0ldGtUC9UR6DXBPa08pac+wO//l2uamaCcKrNNXO1EoYDUM8dKmDLeM=
Message-ID: <10845a340607150515x8b8e1a0k14ae52854ac4fc59@mail.gmail.com>
Date: Sat, 15 Jul 2006 13:15:15 +0100
Reply-To: RQuadling@GoogleMail.com
To: internals@lists.php.net
In-Reply-To: <1677951797.20060715123535@marcus-boerger.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <44B7F0E8.1080001@schlitt.info>
	 <1677951797.20060715123535@marcus-boerger.de>
Subject: Re: [PHP-DEV] Tiny pathinfo() patch
From: rquadling@googlemail.com ("Richard Quadling")

I'm annoyed! Quite annoyed!!

Ok. I know none of you know me and I've not met any of you.

But, a while ago I submitted a relatively minor patch to extend PHP's
php.ini filename to include the major version. This was an extension
to the existing searching rules. As a windows user, I'm not in a
position to build my own executables. I wish I was!!!!

The patch would allow different versions of PHP to use the same SAPI
but with different php.ini files. e.g. originally just php.ini, then
php-cli.ini, php-cgi.ini, php-isapi.ini, etc.

My patch would extend this to _ALSO_ allow php4-cli.ini, php5-cgi.ini,
php6-ispai.ini (just examples - all combinations allowed). And the
patch, whilst untested by me, would allow upto php V99. (I've been
with PHP since just into V4, and soon V6 will be here, so PHP V9
seemed too short sighted).

As this patch has REAL advantages for me and others wishing to use
PHP4 / 5 and 6 on the same machine, with the same webserver (Either
ISAPI OR CGI), I was surprised when it wasn't committed.

Now the patch provided by Tobias, for what is, from his own words,
completely possible in userland without any extension or enhancement
to php core, seems totally redundant.

Whilst I understand that you cannot have every patch supplied
incorporated into the source, would you please comment as to why my
patch was rejected?

Are there some more hoops I'm expected to jump through?

I know that there is a major focus on Unicode at the moment and this
is taking a lot of time. This is why I though actually supplying a
patch would be better received than simply a WIBNI approach.

My intention here is NOT to upset anyone, especially Tobias. He had a
problem and he fixed it and with the support of the core devlopers,
his solution has been incorporated into the core.

I have a different problem. I did the research and supplied a patch.
The patch has been rejected. I do not know why.

If there is more that I need to do to get this functioning, then a few
pointers would be nice.

Regards and true appreciation of the skills, hard work and dedication
of the efforts of the core php developers. I make my living standing
on your shoulders.

Richard Quadling.

P.S. I REALLY don't want to annoy anyone. Really.


On 15/07/06, Marcus Boerger <helly@php.net> wrote:
> Hello Tobias,
>
>   looks good after the update. Can you provide a patch for HEAD too?
>
>   also please provide patches with "cvs di -upd".
>
> best regards
> marcus
>
> Friday, July 14, 2006, 9:30:48 PM, you wrote:
>
> > Hi all!
>
> > It occured several times in the past weeks, where I needed the filename
> > without extension as well as the file extension extracted from a
> > pathname. Since first calling pathinfo() and then basename() again, or
> > using string manipulation, I now patched pathinfo() to provide the
> > filename without extension per default with key "filename".
>
> > Find the patch here: http://schlitt.info/misc/pathinfo.patch
>
> > The patch is against 5_2.
>
> > Maybe someone considers that useful as well?
>
> > Regards,
> > Toby
> > --

-- 
-----
Richard Quadling
Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&amp;r=213474731