Thanks Bob,
I discovered this when using Mark's suggestion.
[snip]
You haven't read a row from SITELOG. So the regex below is checking the
row
from your SITES file.

> 
> # Check the file for any ORA- entries
> 
>     if($_ =~ m/ORA-/) {
> 
>         # If it contains records, reverse the file
> 
>         my @lines = reverse($_);

The call to reverse() is doing nothing here. You're just assigning the
scalar $_ to the array @lines, which will have one element following the
assignment.

Are you just trying to report the ORA-XXX errors in reverse order? If
so,
something like this should work:

    my @errs;
    while (<SITELOG>) {
        push @errs, $_ if /ORA-/;
    }
    print reverse @errs;

Thanks for this, it is neater than my solution!

Regards
Denham

--
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
<http://learn.perl.org/> <http://learn.perl.org/first-response>


Reply via email to