Bug#595899: munin-node: snort_* scripts fail to 'config' due to bashism

2010-09-09 Thread Tom Feiner
tags 595899 pending quit Hi Gerald, I've applied your patch in the following places: * Munin 1.4 stable branch [1] * Munin trunk (2.0): [2] * The next munin debian package which will be uploaded for squeeze [3]. Thanks for the patch Tom Feiner [1] http://munin-monitoring.org/changeset/3884 [2]

Bug#594695: Bug#595899: munin-node: snort_* scripts fail to 'config' due to bashism

2010-09-07 Thread Gerald Turner
Holger Levsen writes: > Hi Gerald, > > thanks for your bugreport, the changes needed are small enough so that > we can still fix them despite the freeze. Also having things work with > dash is a releasegoal :) > Excellent. BTW, I believe bug #594695 (fw_conntrack) is also a bashism, at least sw

Bug#595899: munin-node: snort_* scripts fail to 'config' due to bashism

2010-09-07 Thread Holger Levsen
tags 595899 + patch thanks Hi Gerald, thanks for your bugreport, the changes needed are small enough so that we can still fix them despite the freeze. Also having things work with dash is a releasegoal :) cheers, Holger signature.asc Description: This is a digitally signed message

Bug#595899: munin-node: snort_* scripts fail to 'config' due to bashism

2010-09-06 Thread Gerald Turner
Package: munin-node Version: 1.4.5-2 Severity: minor Hello, the six snort plugins don't work when /bin/sh is dash, they all contain the following code: if [ -n $warning ]; then echo 'alerts.warning $warning' fi if [ -n $critical ]; then echo