[prev in list] [next in list] [prev in thread] [next in thread] 

List:       majordomo-workers
Subject:    problem with config-test.pl, and fix to repair same.
From:       c1040 () azfms ! com (Rusty Carruth)
Date:       1997-07-31 21:20:49
[Download RAW message or body]


I was having some problems getting majordomo working.

The short version is, email to any mailing lists I'd set up
resulted in infinite bounce loop, which grew very quickly
into a full bounce storm.

Looked at permissions and all, and config-test said all was ok,
but I'd made the Log be a directory instead of a file, and config-test
only checks for existance and writability, not for flatness.

The following change to the test script (config-test.pl) corrects this:

Change

if ( ! -e $log ) {
    print "Logfile $log didn't exist, trying to create...\n ";
    if (open(A, ">$log") && close(A) ) { # sesame
	print "okay, now chmod'ing..\n";
	chmod (0664, $log) || &bad( "chmod on $log failed, $!");
    } else {
	&bad("Couldn't create logfile $log, $!\n");
    }
} 


to:

if ( ! -e $log ) {
    print "Logfile $log didn't exist, trying to create...\n ";
    if (open(A, ">$log") && close(A) ) { # sesame
	print "okay, now chmod'ing..\n";
	chmod (0664, $log) || &bad( "chmod on $log failed, $!");
    } else {
	&bad("Couldn't create logfile $log, $!\n");
    }
} else {
   if ( -d $log ) {
	&bad("Bad news, logfile $log is a directory, must be a file!\n");
   }
}

I can create a normal 'diff' or context diff output if you'd rather...

rusty@azfms.com

[prev in list] [next in list] [prev in thread] [next in thread]