|
Harald Hoyer |
53e990 |
From 6f006ae0c1efbbb90d00cbb340001e74d8d12db1 Mon Sep 17 00:00:00 2001
|
|
Harald Hoyer |
53e990 |
From: Harald Hoyer <harald@redhat.com>
|
|
Harald Hoyer |
53e990 |
Date: Fri, 22 Jun 2012 15:34:24 +0200
|
|
Harald Hoyer |
53e990 |
Subject: [PATCH] dracut-logger.sh: use (( )) for numeric comparisons
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
---
|
|
Harald Hoyer |
53e990 |
dracut-logger.sh | 20 ++++++++++----------
|
|
Harald Hoyer |
53e990 |
1 file changed, 10 insertions(+), 10 deletions(-)
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
diff --git a/dracut-logger.sh b/dracut-logger.sh
|
|
Harald Hoyer |
53e990 |
index f1b3a7e..a78e18e 100755
|
|
Harald Hoyer |
53e990 |
--- a/dracut-logger.sh
|
|
Harald Hoyer |
53e990 |
+++ b/dracut-logger.sh
|
|
Harald Hoyer |
53e990 |
@@ -116,7 +116,7 @@ dlog_init() {
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
if [ -z "$fileloglvl" ]; then
|
|
Harald Hoyer |
53e990 |
[ -w "$logfile" ] && fileloglvl=4 || fileloglvl=0
|
|
Harald Hoyer |
53e990 |
- elif [ $fileloglvl -gt 0 ]; then
|
|
Harald Hoyer |
53e990 |
+ elif (( $fileloglvl >= 0 )); then
|
|
Harald Hoyer |
53e990 |
__oldumask=$(umask)
|
|
Harald Hoyer |
53e990 |
umask 0377
|
|
Harald Hoyer |
53e990 |
! [ -e "$logfile" ] && >"$logfile"
|
|
Harald Hoyer |
53e990 |
@@ -138,7 +138,7 @@ dlog_init() {
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
- if [ $sysloglvl -gt 0 ]; then
|
|
Harald Hoyer |
53e990 |
+ if (( $sysloglvl >= 0 )); then
|
|
Harald Hoyer |
53e990 |
if ! [ -S /dev/log -a -w /dev/log ] || ! command -v logger >/dev/null
|
|
Harald Hoyer |
53e990 |
then
|
|
Harald Hoyer |
53e990 |
# We cannot log to syslog, so turn this facility off.
|
|
Harald Hoyer |
53e990 |
@@ -148,7 +148,7 @@ dlog_init() {
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
- if [ $sysloglvl -gt 0 -o $kmsgloglvl -gt 0 ]; then
|
|
Harald Hoyer |
53e990 |
+ if (($sysloglvl >= 0)) || (($kmsgloglvl >= 0 )); then
|
|
Harald Hoyer |
53e990 |
if [ -n "$dracutbasedir" ]; then
|
|
Harald Hoyer |
53e990 |
readonly syslogfacility=user
|
|
Harald Hoyer |
53e990 |
else
|
|
Harald Hoyer |
53e990 |
@@ -159,7 +159,7 @@ dlog_init() {
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
local lvl; local maxloglvl_l=0
|
|
Harald Hoyer |
53e990 |
for lvl in $stdloglvl $sysloglvl $fileloglvl $kmsgloglvl; do
|
|
Harald Hoyer |
53e990 |
- [ $lvl -gt $maxloglvl_l ] && maxloglvl_l=$lvl
|
|
Harald Hoyer |
53e990 |
+ (( $lvl > $maxloglvl_l )) && maxloglvl_l=$lvl
|
|
Harald Hoyer |
53e990 |
done
|
|
Harald Hoyer |
53e990 |
readonly maxloglvl=$maxloglvl_l
|
|
Harald Hoyer |
53e990 |
export maxloglvl
|
|
Harald Hoyer |
53e990 |
@@ -275,14 +275,14 @@ _do_dlog() {
|
|
Harald Hoyer |
53e990 |
local lvlc=$(_lvl2char "$lvl") || return 0
|
|
Harald Hoyer |
53e990 |
local msg="$lvlc: $*"
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
- [ $lvl -le $stdloglvl ] && echo "$msg" >&2
|
|
Harald Hoyer |
53e990 |
- if [ $lvl -le $sysloglvl ]; then
|
|
Harald Hoyer |
53e990 |
+ (( $lvl <= $stdloglvl )) && echo "$msg" >&2
|
|
Harald Hoyer |
53e990 |
+ if (( $lvl <= $sysloglvl )); then
|
|
Harald Hoyer |
53e990 |
logger -t "dracut[$$]" -p $(_lvl2syspri $lvl) "$msg"
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
- if [ $lvl -le $fileloglvl -a -w "$logfile" -a -f "$logfile" ]; then
|
|
Harald Hoyer |
53e990 |
+ if (( $lvl <= $fileloglvl )) && [[ -w "$logfile" ]] && [[ -f "$logfile" ]]; then
|
|
Harald Hoyer |
53e990 |
echo "$msg" >>"$logfile"
|
|
Harald Hoyer |
53e990 |
fi
|
|
Harald Hoyer |
53e990 |
- [ $lvl -le $kmsgloglvl ] && \
|
|
Harald Hoyer |
53e990 |
+ (( $lvl <= $kmsgloglvl )) && \
|
|
Harald Hoyer |
53e990 |
echo "<$(_dlvl2syslvl $lvl)>dracut[$$] $msg" >/dev/kmsg
|
|
Harald Hoyer |
53e990 |
}
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
@@ -304,9 +304,9 @@ _do_dlog() {
|
|
Harald Hoyer |
53e990 |
# echo "This is a warning" | dwarn
|
|
Harald Hoyer |
53e990 |
dlog() {
|
|
Harald Hoyer |
53e990 |
[ -z "$maxloglvl" ] && return 0
|
|
Harald Hoyer |
53e990 |
- [ $1 -le $maxloglvl ] || return 0
|
|
Harald Hoyer |
53e990 |
+ (( $1 <= $maxloglvl )) || return 0
|
|
Harald Hoyer |
53e990 |
|
|
Harald Hoyer |
53e990 |
- if [ $# -gt 1 ]; then
|
|
Harald Hoyer |
53e990 |
+ if (( $# > 1 )); then
|
|
Harald Hoyer |
53e990 |
_do_dlog "$@"
|
|
Harald Hoyer |
53e990 |
else
|
|
Harald Hoyer |
53e990 |
while read line; do
|