[rancid] Re: jerancid show version parsing

[Gino Doucet]

Hi John,
Actually, I'm using 2.3.2a7 version of rancid but verified jerancid in the
final 2.3.2 release to be the same for this segment of code as far as I can
tell.

Cheers,
Gino.


-----Original Message-----
From: 'john heasley' [mailto:heas at shrubbery.net] 
Sent: June 10, 2009 6:11 PM
To: Gino Doucet
Cc: 'Mack, David A (Dave)'; rancid-discuss at shrubbery.net; 'john heasley'
Subject: Re: [rancid] Re: jerancid show version parsing

Wed, Jun 10, 2009 at 12:56:38PM -0300, Gino Doucet:
> Dave/John,
> Not sure if you ever got this resolved but I just noticed it recently and
> realized that it affects us as well.  I made some further modifications to
> the same line of code that John had initially attempted.  However, here is
> what I ended up successfully testing with:
> 
>  -	if (/^(\d+ +\S+ +(\S+).*) \S+$/ && $slots) {
>  +	if (/^(\d+ +\w+ +(\S+).*) (\S+| -+ +)$/ && $slots) {
> 
> As for the slot uptime column I would suggest against including it in your
> rancid reports as it continuously changes.  However, if you really want to
> have it included and cause a version update every time you run rancid
> against your ERX's, modify the line following the above If statement as
> follows:
> 
> >-  	    my($line) = $1;
> >+  	    my($line) = $_;

I believe that I've already fixed this in 2.3.2.