0

Estoy haciendo un script en bash para configurar automáticamente la red. Cuando lo acabé lo metí en https://www.shellcheck.net/ y no me dio ningún fallo. Sin embargo, al ejecutar el scritp me da error en la línea 58. El error es este:

configred.sh: 58: [: si: unexpected operator

Mi código es este:

    #!/bin/bash
    echo "Bienvenido al configurador de red automático"
    echo "Selecciona la red en el la que estás 10.0.0.0 por ejemplo"
    read red
    # El parámetro -z comprueba si la variable está vacía
    if [ -z $red ]
        then
            echo "Debes introducir la dirección de red"
    else
            echo "Ahora introduce la dirección ip de la máquina"
            read ip
            if [ -z $ip ]
                then
                    echo "Debes introducir la máscara"
                    exit
            else
                echo "Introduce la máscara de red"
                read netmask
                if [ -z $netmask ]
                    then
                        echo "Debes introducir una mascara de red"
                        exit
                else
                    echo "Introduce la dirección del servidor DNS por ejemplo 8.8.8.8"
                    read dnsnameserver
                    if [ -z $dnsnameserver ]
                        then
                            echo "Debes introducir la dirección del servidor dns"
                            exit
                    else
                        echo "Introduce el dominio"
                        read search
                        if [ -z "$search" ]
                            then
                                echo "Debes introducir el dominio"
                                exit
                        else
                            echo "Introduce la puerta de enlace"
                            read puerta
                            if [ -z $puerta ]
                                then
                                    echo "Debes introducir una puerta de enlace"
                                    exit
                             else
                                echo "Hacemos copia de seguirdad (el fichero será interfaces2)"
                                    cp /etc/network/interfaces /etc/network/interfaces2
                                echo "auto enp0s3" >> /etc/network/interfaces
                                echo "iface enp0s3 inet static" >> /etc/network/interfaces
                                echo "address $ip" >> /etc/network/interfaces
                                echo "netmask $netmask" >> /etc/network/interfaces
                                echo "network $red" >> /etc/network/interfaces
                                echo "gateway $puerta" >> /etc/network/interfaces
                                echo "dns-nameservers $dnsnameserver" >> /etc/network/interfaces
                                echo "dns-search $search" >> /etc/network/interfaces
                                echo "Configuración acabada"
                                    echo "¿Quieres modificar ahora el fichero resolv.conf? si/no"
                                    read respuestaa
#la línea de abajo es la que falla
                                    if [ "$respuestaa" == "si" ]
                                        then
                                            echo "Procedemos a modificar el fichero /etc/resolv.conf"
                                            sed -i.back '1s/^/ $ip   $HOSTNAME.$dominio  $HOSTNAME\n/' /etc/resolv.conf
                                            echo "Se ha generado un fichero .bak de copia de seguridad"
                                            ls *.bak /etc
                                            echo "¿Quieres reiniciar la tajeta de red para aplicar los cambios? si o no"
                                            read respuesta
                                            if [ "$respuesta" == "si" ] 
                                                then
                                                    ifdown enp0s3
                                                    ifup enp0s3
                                                    echo "se ha reiniciado la tarjeta de red"
                                            elif [ "$respuestaa" == "no" ]
                                                then
                                                    echo "Hasta luego"
                                                    exit
                                            else
                                            echo "Introduce una respuesta válida"
                                            exit
                                            fi
                                    fi
                             fi
                        fi
                    fi
                fi
            fi
    fi
6
  • 1
    Te aconsejo que señales exactamente en que línea tienes el problema, y no solamente el número de línea.
    – Madh
    Commented el 7 may. 2017 a las 12:09
  • 1
    Tal y como comenta @Madh, indica dónde está el error. Además, estás copiando muchísimas líneas para indicar algo que corresponde a una sola. Con un ejemplo mínimo verificable estaríamos mejor. La comparación if [ "$cadena" == "loquesea" ] es correcta, parece que hay algún otro error que no se observa en el código. Ah, hacer un if [ ] -> si correcto, otro if, etc. es un poco peñazo. Sería más útil utilizar if [ error ]; exit y así siempre estar en el mísmo nivel, no en un código spaghetti algo liante.
    – fedorqui
    Commented el 7 may. 2017 a las 13:14
  • Ya está indicado
    – ras212
    Commented el 7 may. 2017 a las 13:40
  • El caso es que hay gente a la que sí le funciona
    – ras212
    Commented el 7 may. 2017 a las 13:48
  • 1
    Tal y como comenté antes, intenta proveer un ejemplo mínimo verificable. Esto incluye los pasos de reproducción del error (¿qué valor le das a $respuestaa). Nótese también que usas respuestaa y respuesta (dos y una as), no sé si con total control sobre cuál hace qué.
    – fedorqui
    Commented el 7 may. 2017 a las 13:50

1 respuesta 1

1

He replanteado el código de forma más clara, y ya es funcional. El código correcto es el siguiente:

#!/bin/bash
echo "Bienvenido al configurador de red automático"
echo "Selecciona la red en el la que estás 10.0.0.0 por ejemplo"
    read red
echo "Ahora introduce la dirección ip de la máquina"
    read ip
echo "Introduce la puerta de enlace"
    read puerta
echo "Introduce la máscara de red"
    read netmask
echo "Introduce la dirección del servidor DNS por ejemplo 8.8.8.8"
    read dnsnameserver
echo "Introduce el dominio"
    read search
# El parámetro -z comprueba si la variable está vacía y -n si está completa
if [ -z $red ] || [ -z $ip ] || [ -z $netmask ] || [ -z $dnsnameserver ] || [ -z "$search" ] || [ -z $puerta ]
    then
        echo "Debes introducir todos los parámetros"
        exit
else
    echo "Hacemos copia de seguirdad (el fichero será interfaces2)"
            cp /etc/network/interfaces /etc/network/interfaces2
    echo "auto enp0s3" >> /etc/network/interfaces
    echo "iface enp0s3 inet static" >> /etc/network/interfaces
    echo "address $ip" >> /etc/network/interfaces
    echo "netmask $netmask" >> /etc/network/interfaces
    echo "network $red" >> /etc/network/interfaces
    echo "gateway $puerta" >> /etc/network/interfaces
    echo "dns-nameservers $dnsnameserver" >> /etc/network/interfaces
    echo "dns-search $search" >> /etc/network/interfaces
    echo "Configuración acabada"

    echo "¿Quieres modificar ahora el fichero resolv.conf? Pulsa una tecla"
        read respuestaa
            if [ -n "$respuestaa" ]
                then
                    echo "Procedemos a modificar el fichero /etc/resolv.conf"
                    sed -i.back '1s/^/ $ip   $HOSTNAME.$dominio  $HOSTNAME\n/' /etc/resolv.conf
                    echo "Se ha generado un fichero .bak de copia de seguridad"
                    ls *.bak /etc
                    echo "¿Quieres reiniciar la tajeta de red para aplicar los cambios? pulsa una tecla para confirmar"
                    read respuesta
                    if [ -n "$respuesta" ]
                        then
                            ifdown enp0s3
                            ifup enp0s3
                            echo "se ha reiniciado la tarjeta de red"
                    elif [ -z "$respuesta" ]
                        then
                            echo "Hasta luego"
                                exit
                    fi
                                   else
                                        echo "Introduce una respuesta válida"
                                        exit
            fi
fi
8
  • ¿Esto quiere decir que ya corregiste el error en el enlace que muestras?
    – fedorqui
    Commented el 7 may. 2017 a las 14:38
  • Sí, ahora falta corregir un problemilla con sed, pero ya tengo una pregunta abierta para ello
    – ras212
    Commented el 7 may. 2017 a las 14:47
  • Sí, esa te le contesté yo también :)
    – fedorqui
    Commented el 7 may. 2017 a las 14:48
  • En cuanto a esta pregunta, incluye la parte importante del código que funciona. De esta forma la respuesta puede ser útil para otros que la lean en el futuro.
    – fedorqui
    Commented el 7 may. 2017 a las 14:49
  • 1
    ras212 no agreges el código usando enlaces externos, estos pueden caer en el futuro y la respuesta sería inútil. Además, esto dificulta la lectura por parte de otros usuarios al tener que salir a páginas externas. He editado tu respuesta pero estaría bien que añadieras alguna explicación al código para añadirle utididad. Saludos.
    – FJSevilla
    Commented el 7 may. 2017 a las 16:43

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