Skip Menu |

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 26977
Status: resolved
Priority: 0/
Queue: Module-Build
People
Owner: Nobody in particular
Requestors: leonerd-cpan [...] leonerd.org.uk
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.2808
Fixed in: (no value)

History Show all quoted text
  Mon May 07 19:54:11 2007 leonerd-cpan [...] leonerd.org.uk - Ticket created
Subject: Fails to check result from Module::Build::Base->check_prereq()
The ->check_prereq() method in Module::Build::Base checks that all the module's build and runtime dependencies are present; returning 0 or 1 to indicate success. This return value is ignored by Module::Build::Base->new(). Is this correct? It means that any missing dependencies are effectively ignored, at least from point of view of any automated controlling program around the build script - only a human operator reading the terminal would notice. -- Paul Evans
  Mon May 07 23:09:28 2007 kwilliams [...] cpan.org - Correspondence added
From: KWILLIAMS [...] cpan.org
Hi Paul, Yeah, that's correct - my preference if I could design everything from scratch might be to throw an exception, but that doesn't jibe well with the existing installation tools. The proper way to check prerequisites is to call the prereq_failures() method and see whether it returns anything. If so, in the return value you also have the information you'll need to try to satisfy them. -Ken
  Mon May 07 23:09:29 2007 The RT System itself - Status changed from 'new' to 'open'
  Mon May 07 23:11:09 2007 kwilliams [...] cpan.org - Status changed from 'open' to 'resolved'