[Ovirt-devel] Re: [PATCH node] ovirt_store_config: return error code

Alan Pevec apevec at redhat.com
Thu Feb 12 14:06:10 UTC 2009


Alan Pevec wrote:
>                  && mount -n --bind /config$f $f \
>                  || printf " Failed to persist\n"
>                  printf " File persisted\n"
> +                rc=1

this is, of course, wrong, correct is:
-                && mount -n --bind /config$f $f \
-                || printf " Failed to persist\n"
-                printf " File persisted\n"
+                && mount -n --bind /config$f $f
+                if [ $? -ne 0 ]; then
+                    printf " Failed to persist\n"
+                    rc=1
+                else
+                    printf " File persisted\n"
+                fi




More information about the ovirt-devel mailing list