Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:51558
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain mozo.jp from 209.85.161.42 cause and error)
MIME-Version: 1.0
In-Reply-To: <4D6F5C79.5070104@gmail.com>
References: <AANLkTinAt4ToUv-FtOeSjbw4Db9iFitE7VvRNZF8yNAc@mail.gmail.com>
 <4D6ECD36.1070804@gmail.com> <AANLkTi=OW9EoTeqOO0vp3xO3YibZ5k4eP+am2B-TW7cz@mail.gmail.com>
 <4D6F5C79.5070104@gmail.com>
Date: Thu, 3 Mar 2011 18:44:46 +0900
Message-ID: <AANLkTin2GeL3drD4p4vngyWbf=a4-BHkr2Tand0=2W9W@mail.gmail.com>
To: =?ISO-8859-1?Q?=C1ngel_Gonz=E1lez?= <keisial@gmail.com>
Cc: PHP Developers Mailing List <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] RFC: built-in web server in CLI.
From: mozo@mozo.jp (Moriyoshi Koizumi)

Hi,

2011/3/3 =C1ngel Gonz=E1lez <keisial@gmail.com>:
> Moriyoshi Koizumi wrote:
>>> Regarding the patch (https://gist.github.com/835698):
>>> I don't see a switch to disable the internal parse on configure.
>> I don't see any obvious reason it should be able to be turned off
>> through the build option. =A0The only problem is binary size increase,
>> which I guess is quite subtle.
> Seems sufficiently different from normal cli.

I've seen a number of people arguing on the same, but I'd rather have
them both in for the sake of simplicity.  As discussed when the CLI
version of PHP was born, multiple PHP binaries actually have confused
the users to a certain degree.

>>> The patch looks messy as it splits main in two functions, so it gets
>>> hard to follow,
>>> but is probably good overall.
>> Assuming you are mentioning about the option parsing portion of the
>> code, yes, it's a bit messy, but I had to do so because runtime
>> initialization procedure is very different from the ordinary CLI.
> Wasn't critizising you. It's a limitation of unified diffs, which can't s=
ay
> "move this bunch of code 25 lines down". A bit hard to follow, but
> aparently good.

I would have put a whitespace-ignoring diff as well ;-)

>>> Any special reason to disable it on PHP_CLI_WIN32_NO_CONSOLE ?
>> cli-win32 version of PHP doesn't have an associated console and is
>> supposed to use to create applications without console interactions
>> (i.e. GUI). =A0So, It doesn't make sense to enable this feature for it.
> With the embedded web server, the interaction would be done via the brows=
er.

It is not intended to be daemonized at all since it's just a
development web server.  To do more, Apache should work great then.

Regards,
Moriyoshi

>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>