Skip Menu |

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

Report information
The Basics
Id: 26931
Status: resolved
Priority: 0/
Queue: Module-Build

Owner: Nobody in particular
Requestors: mca1001 [...]

Bug Information
Severity: (no value)
Broken in:
  • 0.2806
  • 0.2808
Fixed in: (no value)

Subject: process_PL_files doesn't catch script failure [with fix]
Download (untitled) / with headers
text/plain 918b
I noticed that when process_PL_files runs a script and the script fails, the build continues. This seems dangerous... I modified my setup to die when the script fails - we have a subclass setup, so my bodge is a method override plus "is it OK to do that to this version?" safety check. Here's the diff with complications removed. Perhaps you could roll it into the next version? Thanks, Matthew #8-) -- --- 2007-05-04 17:00:34.250539792 +0100 +++ lib/Module/Build/ 2007-05-04 16:44:12.998579527 +0100 @@ -1,14 +1,14 @@ sub process_PL_files { my ($self) = @_; my $files = $self->find_PL_files; while (my ($file, $to) = each %$files) { unless ($self->up_to_date( $file, $to )) { - $self->run_perl_script($file, [], [@$to]) or die "$file failed"; + $self->run_perl_script($file, [], [@$to]); $self->add_to_cleanup(@$to); } } }
Thanks, applied. -Ken

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to