Enhancement: Allow custom "Known error"

[BadWolf42]

Hello,

It would be nice to have the possibility to handle our own known errors as you do for apt, npm, etc.

For exemple, with a function like:
handle <error to grep> <found message> <command/function>
with in parameter the text error to grep in the log, the message to display when found and the command to fixit
it could be declared between each step (or each try ?) to be specific to a stp (or try) or before pre to be generic.

What do you think about it?

Bad

[NebzHB]

Nice idea ! i like it !

handlers would need to be defined before "post" i guess, so anywhere between the "include" of the lib and the "post"

i'll think about it...

[NebzHB]

if you feel comfortable with bash, don't hesitate to PR ;)

[NebzHB]

here is my poc :

#preparation
touch myfile.txt
echo "hello world">myfile.txt

#in dependencies.sh
declare -A fix_handler
h=0
add_fix_handler () {
    fix_handler[$h,search]=$1
    fix_handler[$h,found]=$2
    fix_handler[$h,fix]=$3
    ((h++))
}
fix () {
    for (( i=0;i<h;i++ )); do
        echo "searching for ${fix_handler[$i,search]} in myfile.txt..."
        grep ${fix_handler[$i,search]} myfile.txt &>/dev/null
        if [ "$?" -eq 0 ];then
            echo ${fix_handler[$i,found]}
            ${fix_handler[$i,fix]}
        else
            echo "not found"
        fi
    done
}
post() {
    # ....
    fix
}


# in your dependencies
myfixWorld () {
    echo "fix my world"
}

myfixUniverse () {
    echo "fix my universe"
}

add_fix_handler "universe" "found my universe" myfixUniverse
add_fix_handler "world" "found my world" myfixWorld

post

[NebzHB]

result :

searching for universe in myfile.txt...
not found
searching for world in myfile.txt...
found my world
fix my world

[NebzHB]

Did you had a chance to test it ?

[BadWolf42]

Sorry not much time this week...

[BadWolf42]

Hello I just did a pre-stable dep-install check and got only errors:

''/..': No such file or directoryjMQTT/resources'$'
: No such file or directory/core/class/../../resources/install_apt.sh: line 8: /var/www/html/plugins/jMQTT/resources
': command not founds/jMQTT/core/class/../../resources/install_apt.sh: line 10: $'
': command not founds/jMQTT/core/class/../../resources/install_apt.sh: line 11: $'pre
': command not founds/jMQTT/core/class/../../resources/install_apt.sh: line 12: $'
/var/www/html/plugins/jMQTT/core/class/../../resources/install_apt.sh: line 64: syntax error: unexpected end of file

Dep file is: https://github.com/Domochip/jMQTT/blob/73cd97088abc02cad65dd6662798c50c42348d0e/resources/install_apt.sh

[BadWolf42]

Please ignore, file was in WINDOWS vs Linux file format expected by bash...