On Thu, Apr 09, 2009 at 07:42:27PM -0400, Jason Crawford wrote:
> On Thu, Apr 9, 2009 at 6:52 PM, Marc Espie <es...@nerim.net> wrote:
> > On Thu, Apr 09, 2009 at 05:55:24PM -0400, Jason Crawford wrote:
> >> Yes that sounds like a good idea. So here goes again, attempt number 3.
> >>
> >> -RUN_DEPENDS= ::converters/p5-JSON \
> >> - ::databases/p5-DBI \
> >> - ::databases/p5-DBD-SQLite \
> >> - ::databases/p5-DBD-mysql \
> >> - ::databases/p5-DBD-Pg \
> >> - ::devel/p5-FreezeThaw \
> >> +RUN_DEPENDS= ::devel/p5-FreezeThaw \
> >> ::devel/p5-YAML-Syck \
> >> ::www/p5-CGI-Simple
> >
> > Apparently, json support got dropped ? what about yaml ?
> >
> >> @@ -17,9 +17,7 @@ ${P5SITE}/CGI/Session/ID/static.pm
> >> ${P5SITE}/CGI/Session/Serialize/
> >> ${P5SITE}/CGI/Session/Serialize/default.pm
> >> ${P5SITE}/CGI/Session/Serialize/freezethaw.pm
> >> -${P5SITE}/CGI/Session/Serialize/json.pm
> >> ${P5SITE}/CGI/Session/Serialize/storable.pm
> >> -${P5SITE}/CGI/Session/Serialize/yaml.pm
> >> ${P5SITE}/CGI/Session/Test/
> >> ${P5SITE}/CGI/Session/Test/Default.pm
> >> ${P5SITE}/CGI/Session/Tutorial.pm
> >
> > You should explain what's going on, and why the update is still a good
> > thing ?
> >
> > If I have an app deployed that uses CGI::Session + json, what am I supposed
> > to do ? Why did you remove the json dependency and kept the yaml one ?
> >
> > I could download the tarball, and have a look at what's going on, but the
> > fact you don't talk about it worries me: did you do your job as maintainer ?
> >
>
> Right, sorry about that. According to the Changes file, JSON was
> removed in 4.29_1, and won't be coming back until the code can be made
> to work reliably. As far as YAML, the Changes file states that it was
> moved into it's own perl module, also starting 4.29_1. It was left in
> RUN_DEPENDS purely as an accident, and should be removed.
> As i had some prelimary diffs floating around and thus looked at it too, here's the p5-CGI-Session-Serialize-yaml port. Maybe someone should just take the old json code and make it a new cpan module, even if it has a warning about its drawbacks... code should just continue running after an update if possible. Regards, Simon
p5-CGI-Session-Serialize-yaml.tar.gz
Description: application/tar-gz
