in reply to Re: file lock error
in thread file lock error

ok thank you but I am still getting the same error.

new code 

use strict;
use warnings;
use MIME::Lite;
use FileHandle;
use File::Copy;
use Logfile::Rotate;
use Fcntl qw(:flock);


flock($RLOG, LOCK_SH|LOCK_NB) || warn "no NB lock $!";
flock($MLOG, LOCK_SH|LOCK_NB) || warn "no NB lock $!";

[download]

well similar error NOW: error: can not open: (1)

before :error: can not lock open: (1)"

Replies are listed 'Best First'.
Re^3: file lock error
by ikegami (Patriarch) on Apr 06, 2011 at 20:15 UTC
    As you should. You're locking the file for reading, which prevents Logfile::Rotate from getting an exclusive lock on the file. Why are you locking the file?
[reply]
      I did not intentionally lock the file. Rather I only add the 'flock' calls after I got this error after adding add'l code to my logfile rotate sub.

      I also told logfile rotate Flock => no, but that does not rid of my error.

      Here is my logfile rotate code: 
      sub _logroll($;$) {

    my $gzip;
    eval qq(use Compress::Zlib);
        if ($@) {
            $gzip = q(/usr/bin/gzip);
        }
        else {
            $gzip = q(lib);
        }

    if ( scalar @_ == 1 ) {
        my $roll1 = @_;
        my $rollog1 = new Logfile::Rotate(
           File     => $roll1,
           Count    => 15,
           Dir      => $dir,
           Gzip     => $gzip,
           Flock    => 'no',
           Persist  => 'yes'
        );
        $rollog1->rotate();
    }
    elsif ( scalar @_ == 2 ) {
        my ($roll1, $roll2) = @_;
        my $rollog2 = new Logfile::Rotate(
           File     => $roll1,
           Count    => 15,
           Dir      => $dir,
           Gzip     => $gzip,
           Flock    => 'no',
           Persist  => 'yes'
        );

        my $rollog2a = new Logfile::Rotate(
           File      => $roll2,
           Count     => 15,
           Dir       => $dir,
           Gzip      => $gzip,
           Flock     => 'no',
           Persist   => 'yes'
        );
        
        $_->rotate() for ($rollog2, $rollog2a);
    }
}

      [download]
      later in my code I am '>>' to this same file from a system command called mksysb, so that looks like:

       system ("mksysb /dev/$rmt -A -e -m -p -X >> $mksysblog 2>&1");

[reply]
[d/l]
[select]

        I also told logfile rotate Flock => no, but that does not rid of my error.

        That can't be. That error is only given when Flock is 'yes' or not specified. 

            if( $self->{'Flock'} eq 'yes'){
        $self->{'Fh'} = new IO::File "$self->{'File'}", O_WRONLY|O_EXC
+L;
        croak "error: can not lock open: ($self->{'File'})" 
        unless defined($self->{'Fh'});
        flock($self->{'Fh'},LOCK_EX);
    }

        [download]

        But that doesn't matter. First, don't tell it not to lock.

        Second, it's telling you it's having problem locking file "1". Is your log file really named 1? That's the file name you gave to it. One error is using 

        my $roll1 = @_;

        [download]

        when you mean to use 

        my ($roll1) = @_;

        [download]

        An array in scalar context returns the number of elements in the array.

        Fix: 

        sub _logroll {
    eval 'use Compress::Zlib';
    my $gzip = $@ ? '/usr/bin/gzip' : 'lib';

    for my $log_file ( @_ ) {
        my $rotator = Logfile::Rotate->new(
           File     => $log_file,
           Count    => 15,
           Dir      => $dir,
           Gzip     => $gzip,
           Flock    => 'yes',
           Persist  => 'yes'
        );
        $rotator->rotate();
    }
}

        [download]
[reply]
[d/l]
[select]

In Section Seekers of Perl Wisdom