Skip Menu |

This queue is for tickets about the Graphics-Framebuffer CPAN distribution.

Report information
The Basics
Id: 106851
Status: resolved
Worked: 10 min
Priority: 0/
Queue: Graphics-Framebuffer
People
Owner: RKELSCH [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 5.39

History Show all quoted text
  Thu Sep 03 12:50:59 2015 SREZIC [...] cpan.org - Ticket created
Subject: Can't use 'defined(@array)' (5.39)
Using defined(@array) or defined(%hash) is a hard error in newer perls (e.g. perl 5.22.0) and causes the test suite to fail: # Failed test 'use Graphics::Framebuffer;' # at t/00-load.t line 7. # Tried to use 'Graphics::Framebuffer'. # Error: Can't use 'defined(@array)' (Maybe you should just omit the defined()?) at /home/cpansand/.cpan/build/2015090315/Graphics-Framebuffer-5.39-m0KbAE/blib/lib/Graphics/Framebuffer.pm line 4783. # Compilation failed in require at t/00-load.t line 7. # BEGIN failed--compilation aborted at t/00-load.t line 7. Bailout called. Further testing stopped: # Looks like you failed 1 test of 1. FAILED--Further testing stopped. *** Error code 1
  Thu Sep 03 17:54:28 2015 RKELSCH [...] cpan.org - Correspondence added 10 min
Well that is not cool nor fun. No problem, I modified the code to only check to see if the first element is defined instead of the whole array. This is functionally equivalent to when is needed, so it should be fine on newer Perls. I'm glad it was caught soon after release. Thanks. On Thu Sep 03 12:50:59 2015, SREZIC wrote: Show quoted text
> Using defined(@array) or defined(%hash) is a hard error in newer perls > (e.g. perl 5.22.0) and causes the test suite to fail: > > # Failed test 'use Graphics::Framebuffer;' > # at t/00-load.t line 7. > # Tried to use 'Graphics::Framebuffer'. > # Error: Can't use 'defined(@array)' (Maybe you should just omit > the defined()?) at /home/cpansand/.cpan/build/2015090315/Graphics- > Framebuffer-5.39-m0KbAE/blib/lib/Graphics/Framebuffer.pm line 4783. > # Compilation failed in require at t/00-load.t line 7. > # BEGIN failed--compilation aborted at t/00-load.t line 7. > Bailout called. Further testing stopped: > # Looks like you failed 1 test of 1. > FAILED--Further testing stopped. > *** Error code 1
  Thu Sep 03 17:54:29 2015 The RT System itself - Status changed from 'new' to 'open'
  Thu Sep 03 17:54:30 2015 RKELSCH [...] cpan.org - Status changed from 'open' to 'resolved'
  Thu Sep 03 17:54:31 2015 RKELSCH [...] cpan.org - Taken
  Thu Sep 03 17:54:31 2015 RKELSCH [...] cpan.org - Fixed in 5.39 added