fallos (#4) - Modificar script "deletepreimage" (#397) - Message List

Modificar script "deletepreimage"
 unsolved

Buenos días,

en la instalación de la versión 1.1.0, el script /opt/opengnsys/bin/deletepreimage, en la linea 26, hace un 'mv' sin comprovar antes si existen los ficheros a mover.

Esto provoca que el script genere, en cada ejecución, un email para el administrador de la máquina con el contenido siguiente:

mv: cannot stat '/opt/opengnsys/www/tmp/*.*': No such file or directory

Proponemos modificar el script deletepreimage para hacer un check previo:

26c26 < mv $OPENGNSYS/www/tmp/*.* $OGIMG ---

[ -f $OPENGNSYS/www/tmp/*.* ] && mv $OPENGNSYS/www/tmp/*.* $OGIMG

Gracias, Francesc

  • Message #871

    Hola.

    Le echamos un vistazo para intentar solucionarlo.

    Gracias por el aviso.

  • Message #872

    Hola, Francesc

    No hemos podido reproducir el error. He revisado el script y no entiendo porqué te da el mensaje.

    Antes de la línea que comentas, hay una comprobación de que en el directorio existen ficheros *.delete. En caso contrario se sale del script.

    #TODO en LOCAL: si existe algún fichero *.delete lo movemos al repositorio
    ls $OPENGNSYS/www/tmp/*.delete &>/dev/null || (checkrepo; exit)
    mv $OPENGNSYS/www/tmp/*.* $OGIMG
    

    Siendo la condición de la línea anterior más estricta no debería nunca llegar a intentar mover los ficheros de directorio.

    He probado estas líneas con y sin ficheros *.delete y en ningún caso me da error.

    Si se te ocurre algo más que podamos probar, comentanoslo.

    Hasta ahora

    • Message #873

      Hola Irina,

      gracias por la rapida respuesta (como siempre!).

      Al estar el 'exit' dentro de una subshell (), el codigo principal sigue ejecutandose, no? Una prueba:

      ls $OPENGNSYS/www/tmp/*.delete &>/dev/null || (checkrepo; exit)
      echo "Llego aqui"
      

      No existe el fichero y me imprime "Llego aqui".

      Si es esto, con algo parecido a:

      if [ ! -f $OPENGNSYS/www/tmp/*.delete ]; then
              checkrepo;
              exit;
      fi
      

      Se conseguiría lo que queremos.

      Francesc

Attachments

No attachments created.