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 ?

Reply via email to