[Kolab-devel] [issue2481] debian kdepim patch 12_kmail_clamav_fix.diff

[Bernhard Herzog]

New submission from Bernhard Herzog <bh at intevation.de>:

The debian kdepim enterprise packages we build contain this patch
(12_kmail_clamav_fix.diff):

--- kde.orig/kmail/avscripts/kmail_clamav.sh
+++ kde.patched/kmail/avscripts/kmail_clamav.sh
@@ -35,7 +35,7 @@
 cat > $TEMPFILE
 
 # check for a running daemon
-if [ "`ps -eo comm|grep clamd`" = "clamd" ]; then
+if ps -eo comm | grep -q clamd ; then
     chmod a+r $TEMPFILE
     CLAMCOMANDO="clamdscan --stdout --no-summary "
 else


According to the debian changelog file, this patch was added to fix a debian bug
(http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=354185).  Apparently the
problem was also fixed later in KDE (http://bugs.kde.org/show_bug.cgi?id=126529)
but in a slightly different way and the debian patch was adjusted to the
upstream changes.

I think the debian version of the bug fix is actually better.  Should KDEPIM be
changed accordingly?

----------
assignedto: till
messages: 13710
nosy: bh, ludwig, osterfeld, pradeepto, till, vkrause
priority: minor bug
status: unread
title: debian kdepim patch 12_kmail_clamav_fix.diff
topic: kde client
________________________________________________
Kolab issue tracker <kolab-issues at intevation.de>
<https://intevation.de/roundup/kolab/issue2481>
________________________________________________