On Wed 17 Nov 2004 04:57, "H.Merijn Brand" <[EMAIL PROTECTED]> wrote:
> Return-Path: <[EMAIL PROTECTED]>
> ______________________________________________________________________________
> Automated smoke report for 5.9.2 patch 
> pc03: x86 Family 15 Model 1 Stepping 2, GenuineIntel (x86/1 cpu)
>     on        cygwin_nt-5.0 - 1.5.12(0.116/4/2)
>     using     ? gcc version 3.3.3 (cygwin special)
>     smoketime  (average )
> 
> Summary: PASS
> 
> O = OK  F = Failure(s), extended report at the bottom
> X = Failure(s) under TEST but not under harness
> ? = still running or test results not (yet) available
> Build failures during:       - = unknown or N/A
> c = Configure, m = make, M = make (after miniperl), t = make test-prep
> 
>              Configuration (common) 
> ----------- ---------------------------------------------------------
> | | | +----- PERLIO = perlio -DDEBUGGING
> | | +------- PERLIO = stdio  -DDEBUGGING
> | +--------- PERLIO = perlio
> +----------- PERLIO = stdio
> 
> 
> -- 
> Report by Test::Smoke v1.19 build 777 running on perl 5.9.2
> (Reporter v0.018 / Smoker v0.016)

To prevent reports like this I propose, thouhg using Reporter and the counts
from _parse might be safer:

--- TS-1.19/lib/Test/Smoke/Mailer.pm    2004-04-30 18:24:07.000000000 +0200
+++ smoke/Test/Smoke/Mailer.pm  2004-11-17 10:25:20.523078768 +0100
@@ -122,6 +122,7 @@ sub fetch_report {
     if ( open REPORT, "< $report_file" ) {
         $self->{body} = do { local $/; <REPORT> };
         close REPORT;
+        $self->{body} =~ m/\n-+\s+-+\n\|(\s+\|)*\s+\+--/ and return "";
     } else {
         require Carp;
         Carp::croak "Cannot read '$report_file': $!";
@@ -246,7 +247,7 @@ program.
 sub mail {
     my $self = shift;
 
-    my $subject   = $self->fetch_report();
+    my $subject   = $self->fetch_report() or return $self->{error} = "No test 
results";
     my $cc = $self->_get_cc( $subject );
     my $header = "To: $self->{to}\n";
     $header   .= "From: $self->{from}\n" 
@@ -315,7 +316,7 @@ sub mail {
 
     my $mailer = $self->{mailbin} || $self->{mailxbin};
 
-    my $subject = $self->fetch_report();
+    my $subject = $self->fetch_report() or return $self->{error} = "No test 
results";
     my $cc = $self->_get_cc( $subject );
 
     my $cmdline = qq|$mailer -s '$subject'|;
@@ -385,7 +386,7 @@ sub mail {
 
     $self->{error} = $@ and return undef;
 
-    my $subject = $self->fetch_report();
+    my $subject = $self->fetch_report() or return $self->{error} = "No test 
results";
     my $cc = $self->_get_cc( $subject );
 
     my %message = (
@@ -456,7 +457,7 @@ sub mail {
 
     $self->{error} = $@ and return undef;
 
-    my $subject = $self->fetch_report();
+    my $subject = $self->fetch_report() or return $self->{error} = "No test 
results";
     my $cc = $self->_get_cc( $subject );
 
     my %message = (

-- 
H.Merijn Brand        Amsterdam Perl Mongers (http://amsterdam.pm.org/)
using perl-5.6.1, 5.8.5, & 5.9.x, and 809 on  HP-UX 10.20 & 11.00, 11i,
   AIX 4.3, AIX 5.2, SuSE 9.1, and Win2k.  http://www.cmve.net/~merijn/
http:[EMAIL PROTECTED]/   [EMAIL PROTECTED]
send smoke reports to: [EMAIL PROTECTED], QA: http://qa.perl.org


Reply via email to