Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:61976
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: softfail (pb1.pair.com: domain golemon.com does not designate 209.85.213.42 as permitted sender)
MIME-Version: 1.0
Sender: php@golemon.com
In-Reply-To: <CAFt2z-nHB91SSNaJpn_K1-OzWtC7=W4WUUKCBQYHbb+kbeKsBg@mail.gmail.com>
References: <CANVTSm5g0FWqe=HaJTutniLYOj19rp5twOzF0jti0y3iEsnWEg@mail.gmail.com>
	<CAKOpQSwSwseEoxDYzpfwkgk-azxKyAVq=hRJa=P=v8qg7VAJ_A@mail.gmail.com>
	<CAFt2z-nHB91SSNaJpn_K1-OzWtC7=W4WUUKCBQYHbb+kbeKsBg@mail.gmail.com>
Date: Thu, 2 Aug 2012 16:03:37 -0700
Message-ID: <CAESVnVpXTza67aOOeFqJ72F3=nxe3vTRHWkkzZNc+FChhCbwhg@mail.gmail.com>
To: dukeofgaming <dukeofgaming@gmail.com>
Cc: Kris Craig <kris.craig@gmail.com>, Yahav Gindi Bar <g.b.yahav@gmail.com>, 
	PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] Add runkit to PHP Runtime
From: pollita@php.net (Sara Golemon)

"For all those things you.... probably shouldn't have been doing anyway...."
That description is on the package because you... probably shouldn't
be doing these things.

Further, any discussion of runkit should focus on individual features of it.

Custom Superglobals: I consider these fairly harmless so long as
developers follow strict rules about naming them, but if you want
superglobals, you probably need to rethink some part of your
architecture.

Monkeypatching: Several good use-cases for this (unit tests, among
others), but like all powerful features, must be used with care.

Sandboxing: Complicated by the fact that it only works in a threaded
build, can't transfer all types (e.g. resource, complex objects), and
can't run concurrently.  Until/unless these problems can be
meaningfully solved, I wouldn't consider it a functional
implementation.

For my vote: I think keeping this in pecl is fine, because they're all
things which can be done from the extension space.  Yes, I've
abandoned the package, but nothing is stopping someone else from
taking it over.

-Sara

On Thu, Aug 2, 2012 at 2:28 PM, dukeofgaming <dukeofgaming@gmail.com> wrote:
>
> This is a great idea (the runkit, not the rootkit).
>
> On Thu, Aug 2, 2012 at 1:12 PM, Kris Craig <kris.craig@gmail.com> wrote:
>
> > >
> > > What do you think?
> > >
> >
> > I think I need to get my lens prescription updated.  I thought the subject
> > line read, "Add *rootkit *to PHP Runtime" and was already sharpening my
> > pitchfork before I realized my error.
> >
> > For the record, I am firmly opposed to adding a rootkit to the PHP runtime.
> >  A runkit sounds pretty cool, though.  ;)
> >
> > --Kris
> >