not quite a bug

[Mariano Absatz]

Hi Julian,

browsing the code I found this in Sendmail.pm:

  sub KickMessage {
    my(@ids) = @_;
    my($idlist);

    $idlist = join(' -qI', @ids);
    $idlist .= ' &' if MailScanner::Config::Value('deliverinbackground');
    #print STDERR "About to do \"Sendmail2 -qI$idlist\"\n";
    system(MailScanner::Config::Value('sendmail2') . ' -qI' . $idlist);
  }

and this generates:
sendmail -qI -qIxxxx -qIzzzz -qIwwww

Just for cleanup, I think it should be:
  sub KickMessage {
    my(@ids) = @_;
    my($idlist);

    $idlist = join(' -qI', @ids);
    $idlist .= ' &' if MailScanner::Config::Value('deliverinbackground');
    #print STDERR "About to do \"Sendmail2 -qI$idlist\"\n";
    system(MailScanner::Config::Value('sendmail2') . $idlist);
  }


--
Mariano Absatz
El Baby
----------------------------------------------------------
Errors have been made. Others will be blamed.