Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Mail-DeliveryStatus-BounceParser CPAN distribution.

Report information
The Basics
Id: 64714
Status: resolved
Priority: 0/
Queue: Mail-DeliveryStatus-BounceParser
People
Owner: Nobody in particular
Requestors: MSTEVENS [...] cpan.org
Cc: michael.smith [...] dianomi.com
michael.stevens [...] dianomi.com
AdminCc:
Bug Information
Severity: (no value)
Broken in: 1.525
Fixed in: (no value)

History Show all quoted text
  Wed Jan 12 10:15:33 2011 MSTEVENS [...] cpan.org - Ticket created
CC: michael.stevens [...] dianomi.com,michael.smith [...] dianomi.com
Subject: tests fail on 5.12.2
PERL_DL_NONLAZY=1 /opt/dianomi/perl/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'inc', 'blib/lib', 'blib/arch')" t/*.t t/00-load.t .............................. ok t/aol-attachment.t ....................... ok t/aol-senderblock.t ...................... ok t/aol-vacation.t ......................... ok t/autoreply.t ............................ ok t/bluebottle.t ........................... ok t/email_addr_regex.t ..................... ok t/iis-multiple-bounce.t .................. Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/iis-multiple-bounce.t .................. ok t/malformed-dns.t ........................ ok t/message-too-large.t .................... ok t/misidentified-recipient.t .............. ok t/non-autoreply.t ........................ ok t/not-a-relay.t .......................... ok t/orig-message.t ......................... Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/orig-message.t ......................... 1/12 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/orig-message.t ......................... ok t/pod-coverage.t ......................... skipped: set RELEASE_TESTING t/pod.t .................................. skipped: set RELEASE_TESTING t/polish-autoreply.t ..................... ok t/postfix-malformed.t .................... 1/5 # Failed test 'the right bounced message id is given (but has angle- brackets)' # at t/postfix-malformed.t line 35. ' got: '<42@server4.example.co.uk> # expected: '<42@server4.example.co.uk>' # Looks like you failed 1 test of 5. t/postfix-malformed.t .................... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/5 subtests t/postfix.t .............................. Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/postfix.t .............................. 1/7 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/postfix.t .............................. ok t/quota-2.t .............................. ok t/quota-3.t .............................. ok t/quota-4.t .............................. ok t/quota-5.t .............................. ok t/quota.t ................................ ok t/rcpthosts.t ............................ ok t/relaying-denied.t ...................... ok t/spam-bogus-email-in-report.t ........... ok t/spam-lots-of-bogus-addresses.t ......... ok t/spam-rejection.t ....................... ok t/spam-rejection2.t ...................... ok t/spam-rejection3.t ...................... ok t/spam-rejection4.t ...................... ok t/spam-rejection5.t ...................... ok t/spam-rejection6.t ...................... ok t/spam-rejection7.t ...................... ok t/spam-with-image.t ...................... ok t/spamassassin.t ......................... ok t/spambouncer.t .......................... ok t/status-209.t ........................... Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/status-209.t ........................... ok t/surfcontrol-extra-newline.t ............ ok t/various-domain.t ....................... Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/various-domain.t ....................... 1/18 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/various-domain.t ....................... ok t/various-unknown.t ...................... 1/132 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/various-unknown.t ...................... 39/132 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. Use of uninitialized value in lc at /root/.cpan/build/Mail- DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/various-unknown.t ...................... ok t/virus-caused-multiple-weird-reports.t .. ok t/warnings.t ............................. 1/22 Use of uninitialized value in lc at /root/.cpan/build/Mail-DeliveryStatus-BounceParser-1.525- A3KQrD/blib/lib/Mail/DeliveryStatus/BounceParser.pm line 407. t/warnings.t ............................. ok t/whitelist.t ............................ ok Test Summary Report ------------------- t/postfix-malformed.t (Wstat: 256 Tests: 5 Failed: 1) Failed test: 4 Non-zero exit status: 1 Files=45, Tests=310, 5 wallclock secs ( 0.16 usr 0.05 sys + 4.68 cusr 0.39 csys = 5.28 CPU) Result: FAIL Failed 1/45 test programs. 1/310 subtests failed. make: *** [test_dynamic] Error 255 RJBS/Mail-DeliveryStatus-BounceParser-1.525.tar.gz /usr/bin/make test -- NOT OK //hint// to see the cpan-testers results for installing this module, try: reports RJBS/Mail-DeliveryStatus-BounceParser-1.525.tar.gz Running make install make test had returned bad status, won't install without force Failed during this command: RJBS/Mail-DeliveryStatus-BounceParser-1.525.tar.gz: make_test NO perl is: Summary of my perl5 (revision 5 version 12 subversion 2) configuration: Platform: osname=linux, osvers=2.6.18-194.26.1.el5, archname=x86_64-linux uname='linux 303646-web1.dianomi.co.uk 2.6.18-194.26.1.el5 #1 smp fri oct 29 14:21:16 edt 2010 x86_64 x86_64 x86_64 gnulinux ' config_args='-des -Dprefix=/opt/dianomi/perl -Accflags=-fPIC' hint=recommended, useposix=true, d_sigaction=define useithreads=undef, usemultiplicity=undef useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef use64bitint=define, use64bitall=define, uselongdouble=undef usemymalloc=n, bincompat5005=undef Compiler: cc='cc', ccflags ='-fPIC -fno-strict-aliasing -pipe -fstack- protector -I/usr/local/include -D_LARGEFILE_SOURCE - D_FILE_OFFSET_BITS=64', optimize='-O2', cppflags='-fPIC -fno-strict-aliasing -pipe -fstack-protector - I/usr/local/include' ccversion='', gccversion='4.1.2 20080704 (Red Hat 4.1.2-48)', gccosandvers='' intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678 d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16 ivtype='long', ivsize=8, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8 alignbytes=8, prototype=define Linker and Libraries: ld='cc', ldflags =' -fstack-protector -L/usr/local/lib' libpth=/usr/local/lib /lib /usr/lib /lib64 /usr/lib64 /usr/local/lib64 libs=-lnsl -ldb -ldl -lm -lcrypt -lutil -lc perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc libc=/lib/libc-2.5.so, so=so, useshrplib=false, libperl=libperl.a gnulibc_version='2.5' Dynamic Linking: dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E' cccdlflags='-fPIC', lddlflags='-shared -O2 -L/usr/local/lib -fstack- protector' Characteristics of this binary (from libperl): Compile-time options: PERL_DONT_CREATE_GVSV PERL_MALLOC_WRAP USE_64_BIT_ALL USE_64_BIT_INT USE_LARGE_FILES USE_PERLIO USE_PERL_ATOF Built under linux Compiled at Jan 11 2011 13:45:15 @INC: /opt/dianomi/perl/lib/site_perl/5.12.2/x86_64-linux /opt/dianomi/perl/lib/site_perl/5.12.2 /opt/dianomi/perl/lib/5.12.2/x86_64-linux /opt/dianomi/perl/lib/5.12.2 .
  Wed Jan 12 17:15:01 2011 MSTEVENS [...] cpan.org - Correspondence added
Attaching proposed fixes. As usual also in gitorious: http://gitorious.org/mail-deliverystatus-bounceparser-fork/mail-deliverystatus-bounceparser-fork
Subject: 0001-Fix-the-undefined-variable-warnings.patch
From e59cbc8df4987267658e3e85be291c8b8f81090b Mon Sep 17 00:00:00 2001 From: Michael Stevens <mstevens@etla.org> Date: Wed, 12 Jan 2011 21:55:26 +0000 Subject: [PATCH 1/2] Fix the undefined variable warnings. --- lib/Mail/DeliveryStatus/BounceParser.pm | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/lib/Mail/DeliveryStatus/BounceParser.pm b/lib/Mail/DeliveryStatus/BounceParser.pm index 44c2e20..29fbdda 100644 --- a/lib/Mail/DeliveryStatus/BounceParser.pm +++ b/lib/Mail/DeliveryStatus/BounceParser.pm @@ -404,7 +404,8 @@ sub parse { # Some MTAs send unsought delivery-status notifications indicating # success; others send RFC1892/RFC3464 delivery status notifications # for transient failures. - if (my $action = lc $report->get('Action')) { + if (defined $report->get('Action') and lc $report->get('Action')) { + my $action = lc $report->get('Action'); $action =~ s/^\s+//; if ($action =~ s/^\s*([a-z]+)\b.*/$1/s) { # In general, assume that anything other than 'failed' is a -- 1.7.1
Subject: 0002-Fix-postfix-malformed-on-perl-5.12.2.patch
  Wed Jan 12 17:15:02 2011 MSTEVENS [...] cpan.org - Status changed from 'new' to 'open'
  Wed Jan 12 17:15:53 2011 MSTEVENS [...] cpan.org - Correspondence added
On Wed Jan 12 17:15:01 2011, MSTEVENS wrote: Show quoted text
> Attaching proposed fixes. As usual also in gitorious: > > http://gitorious.org/mail-deliverystatus-bounceparser-fork/mail- > deliverystatus-bounceparser-fork
Hmm where's my second attachment gone.
Subject: 0002-Fix-postfix-malformed-on-perl-5.12.2.patch
From 43403f8ad60af173b6f99fa68ae7ad90fc0c361a Mon Sep 17 00:00:00 2001 From: Michael Stevens <mstevens@etla.org> Date: Wed, 12 Jan 2011 22:05:42 +0000 Subject: [PATCH 2/2] Fix postfix-malformed on perl 5.12.2. --- lib/Mail/DeliveryStatus/BounceParser.pm | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Mail/DeliveryStatus/BounceParser.pm b/lib/Mail/DeliveryStatus/BounceParser.pm index 29fbdda..896f34a 100644 --- a/lib/Mail/DeliveryStatus/BounceParser.pm +++ b/lib/Mail/DeliveryStatus/BounceParser.pm @@ -322,8 +322,8 @@ sub parse { # see MIME::Entity regarding REPLACE my $orig_message_id = $orig_message->parts(0)->head->get("message-id"); if ($orig_message_id) { - chomp $orig_message_id; - $self->log("extracted original message-id $orig_message_id from the original rfc822/message"); + $orig_message_id =~ s/(\r|\n)*$//g; + $self->log("extracted original message-id [$orig_message_id] from the original rfc822/message"); } else { $self->log("Couldn't extract original message-id from the original rfc822/message"); } -- 1.7.1
  Thu Mar 31 21:22:30 2011 rjbs [...] cpan.org - Status changed from 'open' to 'resolved'