0

Tengo el siguiente código:

#!/bin/bash
while true
do
  cat <<-EOF
MENÚ DE COPIAS DE SEGURIDAD
1       Copia manual de un directorio
2       Copia manual de un fichero
3       Añadir tareas a CRON
4       Ver lista de tareas automatizadas de CRON
5       Salir
EOF
  read respuesta

  case $respuesta in
    1)  echo "Introduzca el directorio de origen"
                read directorio
        echo "Introduzca la ruta de destino"
                read destino
                        if [ -d $directorio ] && [ -d $destino ]
                                then
                                        cp -r $directorio $destino
                        else
                                echo "Introduzca una ruta válida"
                        fi;;
    2) echo "Introduzca el directorio de origen del fichero"
        read fichero
        echo "Introduzca la ruta de destino"
        read destinof
                if [ -d $fichero ] && [ -d $destinof ]
                        then
                                cp -r $fichero $destinof
                else
                        echo "Se ha equivocado"
                fi;;
    3) echo "Bienvenido a CRON"
        crontab -e;;
    4) echo "Lista de tareas activas:"
        crontab -l;;
    5) echo "Hasta luego"
        exit;;
    *) echo "Debe escoger una opción válida";;
  esac
done

Pero a la hora de ejecutarlo, me da error en línea 29, creo que es por la condición múltiple del IF, pero no estoy seguro. Edito: el error es exactamente este: copiaseguridad.sh: 29: [: "/home/rafa/Documentos/Hacklab: unexpected operator

3
  • En qué linea? Creo que no me falta nada
    – ras212
    Commented el 18 oct. 2016 a las 15:50
  • Utiliza comillas dobles cuando manejes el valor de las variables. Pon [ -d "$fichero" ] && [ -d "$destinof" ]. Comprueba si [ -d 1 2 ] da el mismo mensaje de error que el que obtienes al ejecutar el script..
    – Jdamian
    Commented el 19 oct. 2016 a las 14:35
  • Cuando pueda lo pruebo
    – ras212
    Commented el 20 oct. 2016 a las 17:48

3 respuestas 3

1

Tienes un problema con rutas que contengan espacios en blanco (tanto en el nombre del directorio como del archivo).

Lo puedes solucionar delimitando el contenido de las variables (uses o no la forma que te dice @PaulRM) de la siguiente manera:

[ -d "$fichero" ] && [ -d "$destinof" ]

Ojo, no he corregido el tipo de prueba (-d para comprobar existencia de directorios y -f para archivos).

Igualmente deberías delimitar líneas en las que uses dichas rutas como parámetros, como ésta:

cp -r "$fichero" "$destinof"

Un saludo.

0

Cambia la linea 29

if [ -d $fichero ] && [ -d $destinof ]

a

if [ -f $fichero -a -d $destinof ]
5
  • ambos operadores -a y && son validos. no creo que sea el problema
    – lois6b
    Commented el 18 oct. 2016 a las 15:55
  • No funciona, sigue dando el mismo error
    – ras212
    Commented el 18 oct. 2016 a las 16:03
  • @ras212 no entiendo como tienes el mismo error, pero por otro lado veo que: en la linea 29, preguntas por un fichero con -d if [ -d $fichero ] debería ser if [ -f $fichero ]
    – PaulRM
    Commented el 18 oct. 2016 a las 16:13
  • @PaulRM -d sirve para comprobar que exista el fichero y ademas sea directorio (carpeta).Su variable la llamó fichero (desafortunadamente) porque "Introduzca el directorio de origen del fichero". -f es para comprobar que exista y sea archivo regular. tldp.org/LDP/Bash-Beginners-Guide/html/sect_07_01.html
    – lois6b
    Commented el 19 oct. 2016 a las 6:02
  • Efectivamente, eso es lo que quería decir
    – ras212
    Commented el 20 oct. 2016 a las 17:48
0

Esta es la solución:

#!/bin/bash
while true
do
  cat <<-EOF
MENÚ DE COPIAS DE SEGURIDAD
1   Copia manual de un directorio
2   Copia manual de un fichero
3   Añadir tareas a CRON
4   Ver lista de tareas automatizadas de CRON
5   Salir
EOF
  read respuesta

  case $respuesta in
    1)  echo "Introduzca el directorio de origen"
        read directorio
    echo "Introduzca la ruta de destino"
        read destino
            if [ -d $directorio ] && [ -d $destino ]
                then
                    cp -r $directorio $destino
            else
                echo "Introduzca una ruta válida"
            fi;;
    2) echo "Introduzca el directorio de origen del fichero"
    read fichero
    echo "Introduzca la ruta de destino"
    read destinof
        if [ -f $fichero ] && [ -d $destinof ]
            then
                cp -r $fichero $destinof
        else
            echo "Se ha equivocado"5
        fi;;
    3) echo "Bienvenido a CRON"
    crontab -e;;
    4) echo "Lista de tareas activas:"
    crontab -l;;
    5) echo "Hasta luego"
    exit;;
    *) echo "Debe escoger una opción válida";;
  esac
done

Prueba:introducir la descripción de la imagen aquí

12
  • ¿"Ésta es la solución"? ¿Qué has cambiado para arreglarlo? Commented el 1 nov. 2016 a las 20:50
  • He puesto los parámetros -d y -f bien. ese era el error. No me di cuenta
    – ras212
    Commented el 1 nov. 2016 a las 20:57
  • Es que copié y pegué tu script en mi máquina para reproducir tu problema y únicamente salía tu error si ponía rutas con espacios en blanco (pongo /tmp mal como ruta): pruebas.sh: línea 29: [: /tmp: se esperaba un operador binario. Si hago la modificación que tú dices sigue fallando en exactamente en la misma línea con el mismo mensaje de error. A parte de eso, si fue @PaulRM quien te dio con la solución, deberías haber votado su respuesta como la correcta y darle los puntos de reputación a él. Commented el 2 nov. 2016 a las 8:36
  • Yo puse la ruta entre comillas
    – ras212
    Commented el 4 nov. 2016 a las 19:20
  • ¿Y poniendo la ruta entre comillas eres capaz de reproducir el problema que tenías planteado originalmente? Commented el 4 nov. 2016 a las 20:08

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.