Chris Hagglund wrote:
> Mark Stosberg wrote:
>
>>On 2006-02-24, Chris Hagglund <[EMAIL PROTECTED]> wrote:
>>
>>
>>>[EMAIL PROTECTED] CGI-Application-4.04]# make test
>>>PERL_DL_NONLAZY=1 /usr/local/bin/perl -Iblib/arch -Iblib/lib
>>>-I/usr/lib/perl5/5.6.1/i386-linux -I/usr/lib/perl5/5.6.1 -e 'use
>>>Test::Harness qw(&runtests $verbose); $verbose=0; runtests @ARGV;' t/*.t
>>>t/01cgiapp..................ok
>>>t/02mailform................ok
>>>t/03prerun..................ok
>>>t/04getquery................ok
>>>t/05arrayrefmodes...........ok
>>>t/06enhancement31...........ok
>>>t/07postrun.................ok
>>>t/09zerorm..................ok
>>>t/10errormode...............ok
>>>t/11callbacks...............ok
>>>t/load_tmpl_hook............ok 2/0Error executing run mode 'start':
>>>Illegal division by zero at (eval 4) line 12.
>>>at t/load_tmpl_hook.t line 23
>>># Looks like your test died just after 2.
>>>t/load_tmpl_hook............dubious
>>> Test returned status 255 (wstat 65280, 0xff00)
>>> after all the subtests completed successfully
>>>
>>>
>>
>>That's interesting. Add this to the top of t/load_tmpl_hook.t to help
>>nail down where the error is coming from:
>>
>> use Carp;
>> $SIG{__WARN__} = \&carp;
>> $SIG{__DIE__} = \&confess;
>>
>> Mark
>>
>>
>>
> Thaks Mark, I've added those lines, ran the test again and this is the
> new output from the spot where it fails the test:
>
> t/load_tmpl_hook............ok 2/0Error executing run mode 'start':
> Illegal division by zero at (eval 4) line 12.
> CGI::Dump('CGI=HASH(0x819de9c)') called at
> blib/lib/CGI/Application.pm line 297
> CGI::Application::dump_html('CGI::Application=HASH(0x8173430)')
> called at blib/lib/CGI/Application.pm line 152
> eval {...} called at blib/lib/CGI/Application.pm line 151
> CGI::Application::run('CGI::Application=HASH(0x8173430)') called
> at t/load_tmpl_hook.t line 26
> at t/load_tmpl_hook.t line 26
> Carp::croak('Error executing run mode \'start\': Illegal
> division by zero at ...') called at blib/lib/CGI/Application.pm line 160
> CGI::Application::run('CGI::Application=HASH(0x8173430)') called
> at t/load_tmpl_hook.t line 26
>
> This is what line 297 looks like in Application.pm:
>
> $output .= $query->Dump;
>
> ------------------------------
> Ok so I think maybe I should try to update the CGI module. Done (to
> latest CGI.pm, version 3.17)... Woohoo now this test passes.
>
> But, OH NO! Now a whole bunch of other tests are failing (all seem
> related to expected headers after doing setting a redirect header and
> always getting back a 'Found' instead of a 'Moved'). Seemed pretty
> strange. So after looking at what those tests were actually trying to do
> I decided I'd try downgrading CGI.pm to the newest one before the
> release of cgiapp 4.04 (seemed to be CGI.pm version 3.11), and now all
> tests are passing. I gotta wonder what old version of CGI.pm I had
> before starting all of this haha.
I looked into this some more.
There was a behavior change in 3.16 to become more compliant with a
standard.
I've now addressed this in CGI::Application by updating our tests to expect
the newer standard syntax, and requiring at least 3.16 to insure that
syntax is provided by CGI.pm. This should save some others who follow your
path some grief.
As there are no functionality changes, there is no other reason to update to
4.05.
Mark
---------------------------------------------------------------------
Web Archive: http://www.mail-archive.com/cgiapp@lists.erlbaum.net/
http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
