[Bash] Script de Backup

[notfound]

o/

Déjà, merci pour le partage ! C'est une bonne idée de script
Par contre, il y a quelques défauts mais on va les corriger ensemble.

################################################################################

Tout d'abord, j'ai pas compris cette ligne :

Code BASH :


echo "Veuillez entrer la cause de ce backup :"\n
 

Si tu veux sauter une ligne avec le \n ça ne marchera pas si tu le mets ici !

Code BASH :


# Fonctionnant avec #!/bin/bash
echo -e "Veuillez entrer la cause de ce backup : \n"

# Fonctionnant avec #!/bin/sh
echo "Veuillez entrer la cause de ce backup : \n"
 

###############################################################################

Ensuite, tu as :

Code BASH :


echo -n "cause : "
read choix
 

Que tu peux remplacer par :

Code BASH :

read -p "Cause : " choix

################################################################################

Pour le nom du fichier en fonction de la date

Code BASH :

la_date=$(date +%d/%m/%Y)

Si jamais tu fais 2 backup dans la même journée, ça écrase tout donc c'est pas forcément bien ... Sauf si t'es sûr. Mais bon tu peux préciser avec l'heure :

Code BASH :

la_date=$(date "+%Y/%m/%d_%H:%M")

################################################################################

Pour finir, je vois 2 choses génantes :
- L'utilisation de trop de IF --> CASE ... IN
- Si un user ne mets pas 1, 2 ou 3 tu quittes ton programme, il faut donc le relancer ce qui n'est pas très élégant en soit


Voici mon code corrigé pour SLAANESH_backup.sh :

Code BASH :


#!/bin/bash

echo -e "Veuillez entrer la cause de ce backup : \n"
echo "Backup de routine     (1)"
echo "Veille d'un départ    (2)"
echo "Changement de support (3)"

while true; do
    read -p "Cause : " choix
    case $choix in
        '1')
            cause="Backup de routine effectué"
            break ;;
        '2')
            cause="Veille d'un départ"
            break ;;
        '3')
            cause="Changement de support"
            break ;;
        *)
            echo "Veuillez taper un choix correct après avoir relancé le programme" ;;
    esac
done

la_date=$(date "+%Y/%m/%d_%H:%M")
echo "$cause le $la_date" >> logs.txt
echo "$cause le $la_date" >> backup_SLAANESH/logs.txt
exit 0
 

Concernant le 2e script, tout a été dit par Gruik !

EDIT : Il serait aussi bien cool que la clé USB soit détectée automatiquement, plutôt que ce soit à l'user de changer le script