Skip Menu |

This queue is for tickets about the Net-Pcap CPAN distribution.

Report information
The Basics
Id: 44448
Status: resolved
Priority: 0/
Queue: Net-Pcap
People
Owner: Nobody in particular
Requestors: craig_davison2001 [...] yahoo.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.16
Fixed in: 0.17

History Show all quoted text
  Fri Mar 20 14:50:44 2009 craig_davison2001 [...] yahoo.com - Ticket created
Subject: t/17-lib_version.t not working with opensuse libpcap
t/17-lib_version.t does not work with OpenSUSE 11.1, which ships with a CVS version of libcap (version 0.9-PRE-CVS). Here is a patch to add support for version numbers in that format: diff -U 1 -r Net-Pcap-0.16/t/17-lib_version.t Net-Pcap-0.16-fixed/t/17-lib_version.t --- Net-Pcap-0.16/t/17-lib_version.t 2007-12-22 12:42:24.000000000 -0700 +++ Net-Pcap-0.16-fixed/t/17-lib_version.t 2009-03-20 12:43:50.000000000 -0600 @@ -17,3 +17,3 @@ like( $version, - '/^libpcap version (?:\d\.\d+\.\d+|unknown \(pre 0\.8\))$/', + '/^libpcap version (?:\d\.\d+\.\d+|\d\.\d+[^0-9.\s]*|unknown \(pre 0\.8\))$/', " - checking version string ($version)"
  Fri Mar 20 14:58:07 2009 craig_davison2001 [...] yahoo.com - Correspondence added
Here's a better patch, which replaces my 0-9 with \d: diff -U 1 -r Net-Pcap-0.16/t/17-lib_version.t Net-Pcap-0.16-fixed/t/17-lib_version.t --- Net-Pcap-0.16/t/17-lib_version.t 2007-12-22 12:42:24.000000000 -0700 +++ Net-Pcap-0.16-fixed/t/17-lib_version.t 2009-03-20 12:55:29.000000000 -0600 @@ -17,3 +17,3 @@ like( $version, - '/^libpcap version (?:\d\.\d+\.\d+|unknown \(pre 0\.8\))$/', + '/^libpcap version (?:\d\.\d+\.\d+|\d\.\d+[^\d.\s]*|unknown \(pre 0\.8\))$/', " - checking version string ($version)"
  Fri Mar 20 14:58:07 2009 craig_davison2001 [...] yahoo.com - Status changed from 'new' to 'open'
  Tue Mar 24 18:44:26 2009 SAPER [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #44448] t/17-lib_version.t not working with opensuse libpcap
Date: Tue, 24 Mar 2009 23:44:05 +0100
To: bug-Net-Pcap [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <saper [...] cpan.org>
Craig Davison wrote via RT: Show quoted text
> t/17-lib_version.t does not work with OpenSUSE 11.1, which ships > with a > CVS version of libcap (version 0.9-PRE-CVS). Here is a patch to add > support for version numbers in that format:
Thanks, applied and optimised. Index: t/17-lib_version.t =================================================================== --- t/17-lib_version.t (revision 91) +++ t/17-lib_version.t (working copy) @@ -15,7 +15,7 @@ } else { like( $version, - '/^libpcap version (?:\d+\.\d+(?:\.\d+)?|unknown \(pre 0 \.8\))$/', + '/^libpcap version (?:\d+\.\d+(?:\.\d+|[^\d.\s]+)?| unknown \(pre 0\.8\))$/', " - checking version string ($version)" ); } -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.
  Wed Nov 28 15:51:15 2012 SAPER [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Nov 28 15:51:15 2012 SAPER [...] cpan.org - Fixed in 0.17 added