[Fwd: Re: [PEAR-DEV] Re: [PEAR-CVS] cvs: pear /Services_JSON JSON.php Test-JSON.php]

How you can present things -------- Original Message -------- Subject: Re: [PEAR-DEV] Re: [PEAR-CVS] cvs: pear /Services_JSON JSON.php Test-JSON.php Date: Fri, 24 Mar 2006 17:50:49 +0100 From: toggg@php.net (bertrand Gugger) Reply-To: toggg@php.net To: Daniel Convissor <danielc@analysisandsolutions.com> CC: Michal Migurski <migurski@php.net>, pear-cvs@lists.php.net, PEAR Dev List <pear-dev@lists.php.net>, PEAR QA List <pear-qa@lists.php.net> Newsgroups: php.pear.cvs,php.pear.dev,php.pear.qa References: <cvsmigurski1143179864@cvsserver> <20060324154010.GA1212@panix.com> Bonjour, Daniel Convissor wrote:

On Fri, Mar 24, 2006 at 05:57:44AM -0000, Michal Migurski wrote:

migurski Fri Mar 24 05:57:44 2006 UTC

 Added files:                    /pear/Services_JSON	JSON.php Test-JSON.php    

Cool. Glad to see this getting into CVS! Three of the votes for this package expressed reservations with putting this package being in the Services category: http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=wenz http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=justinpatrin http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=bolk I share those reservations. Perhaps put it in the Text or HTML categories? Please, a consensus on where to put this should be reached before the first release. Thanks, --Dan I was also happy to see this commit.

Congrats Michal ! As for other classes, I really like to see client executable code included. For the category, if javascript belongs to html , then HTML category. That should be, as javascript belongs to public. Just not the server of it. I would add a call for a new role by pear installer , client script , as javascript Should be in some public area. Regards -- toggg