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.
