diff options
author | Dieter Plaetinck <dieter@plaetinck.be> | 2008-11-03 16:39:33 +0100 |
---|---|---|
committer | Dieter Plaetinck <dieter@plaetinck.be> | 2008-11-03 16:39:33 +0100 |
commit | 92379ba136d863158d06b2ebb31b822779adffec (patch) | |
tree | 540d8edfff63bd6ec3bccd3c47f108a693057537 | |
parent | 6bb0e1b842bc3dae69b801112e4cfd39d6947a49 (diff) |
logging function + use it instead of abusing notify for less important things
-rw-r--r-- | src/core/libs/lib-ui.sh | 12 | ||||
-rwxr-xr-x | src/fifa.sh | 18 |
2 files changed, 24 insertions, 6 deletions
diff --git a/src/core/libs/lib-ui.sh b/src/core/libs/lib-ui.sh index ef6ece7..6b8393f 100644 --- a/src/core/libs/lib-ui.sh +++ b/src/core/libs/lib-ui.sh @@ -50,6 +50,18 @@ notify () } +# logging of stuff +log () +{ + if [ "$var_UI_TYPE" = dia ] + then + echo -e "[LOG] `date +"%Y-%m-%d %H:%M:%S"` $@" >$LOG + else + echo -e "[LOG] `date +"%Y-%m-%d %H:%M:%S"` $@" + fi +} + + # ask the user a password. return is stored in $PASSWORD or $<TYPE>_PASSWORD # $1 type (optional. eg 'svn', 'ssh'). ask_password () diff --git a/src/fifa.sh b/src/fifa.sh index 554436d..4d3a5da 100755 --- a/src/fifa.sh +++ b/src/fifa.sh @@ -30,6 +30,12 @@ notify () } +log () +{ + echo -e "[LOG] `date +"%Y-%m-%d %H:%M:%S"` $@" +} + + ###### Core functions ###### @@ -37,7 +43,7 @@ notify () load_module () { [ -z "$1" ] && die_error "load_module needs a module argument" - notify "Loading module $1 ..." + log "Loading module $1 ..." path=/home/arch/fifa/user/"$1" [ "$1" = core ] && path=/home/arch/fifa/core @@ -67,11 +73,11 @@ load_procedure() [ -z "$2" ] && die_error "load_procedure needs a procedure as \$2" if [ "$1" = 'http:' ] then - notify "Loading procedure $2 ..." + log "Loading procedure $2 ..." procedure=/home/arch/fifa/runtime/procedure-downloaded-`basename $2` wget "$2" -q -O $procedure >/dev/null || die_error "Could not download procedure $2" else - notify "Loading procedure $1/procedures/$2 ..." + log "Loading procedure $1/procedures/$2 ..." procedure=/home/arch/fifa/user/"$1"/procedures/"$2" [ "$1" = core ] && procedure=/home/arch/fifa/core/procedures/"$2" fi @@ -85,7 +91,7 @@ load_lib () { [ -z "$1" ] && die_error "load_library needs a module als \$1 and library as \$2" [ -z "$2" ] && die_error "load_library needs a library as \$2" - notify "Loading library $1/libs/$2 ..." + log "Loading library $1/libs/$2 ..." lib=/home/arch/fifa/user/"$1"/libs/"$2" [ "$1" = core ] && lib=/home/arch/fifa/core/libs/"$2" source $lib || die_error "Something went wrong while sourcing library $lib" @@ -96,8 +102,8 @@ execute () { [ -z "$1" -o -z "$2" ] && die_error "Use the execute function like this: execute <type> <name> with type=phase/worker" [ "$1" != phase -a "$1" != worker ] && die_error "execute's first argument must be a valid type (phase/worker)" - [ "$1" = phase ] && notify "******* Executing phase $2" - [ "$1" = worker ] && notify "*** Executing worker $2" + [ "$1" = phase ] && log "******* Executing phase $2" + [ "$1" = worker ] && log "*** Executing worker $2" if type -t $1_$2 | grep -q function then PWD_BACKUP=`pwd` |