Hi,

> +     warning_ck=`echo ${stdouterr} |grep "^W\:"` || [ "$verbose" != "yes" ] 
> || echo "no 'apt-get update' warning"
> +     if [ "${warning_ck}" != "" ]; then
> +             echo "WARNING: $stdouterr"
> +             stdouterr=""
> +     fi

AFAIU, if there is any warning, we clean stdouterr.

Thus we igore any error here if there is any warning (but not if there
is no warning):

>       [ "$stdouterr" = "" ] || die_hard_with_a_cleanup "failed to retrieve 
> status information from google : $stdouterr"

Shouldn't this be something like:

> +     # If there anything other than warnings die:
> +     ( echo "$stdouterr" | grep -v ^W: ) || die_hard_with_a_cleanup "failed 
> to retrieve status information from google : $stdouterr"
> +     # Otherwise echo the warnings:
> +     [ "$stdouterr" = "" ] || echo "WARNING: $stdouterr"

--
Gabriel

Reply via email to