Steve Langasek <[EMAIL PROTECTED]>  wrote:
> Well, hard to say it's related without actually being able to reproduce the
> error, but this:
> 
>                 add_clamav2group || true
> 
> means that any errors from trying to add the clamav user to the clamsmtp
> group on package install are silently ignored.  Probably better to propagate
> the error, then at least this bug report might be debuggable as a package
> failing to install, instead of a package failing to run.

I think this is a good idea.  I wasn't real comfortable adding the "||
true" to begin with, but saw other scripts that did this as
precedence.  Thanks for investigating.

> I can't see any overt reason why it should have failed, though, so I
> wonder if this bug should be downgraded regardless.

I would have to agree here as well.  I will downgrade to normal.  Olaf
had been sending me emails directly regard this bug report that I
could forward to the queue. 

In any case, I'll take the "|| true" off and upload the package today.

-- 
Chad Walstrom <[EMAIL PROTECTED]>           http://www.wookimus.net/
           assert(expired(knowledge)); /* core dump */



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to