Anomalie #2635
Chemins "en dur" dans posttemplate/00-bacula
100%
Description
Il y a 2 occurrences de /mnt/sauvegardes alors que juste au-dessus :
MOUNT_DIR='/mnt/sauvegardes'
Demandes liées
Révisions associées
Utilisation d'une variable plutôt que des valeurs
- eole/posttemplate/00-bacula : remplacement d'une occurence de
/mnt/sauvegardes par $MOUNT_DIR (la seconde occurence a été supprimée
lors du remplacement de umount par baculamount.py --umount). Ajout
d'un avertissement pour cette variable qui ne doit pas variée. En
cause, son utilisation dans plusieurs scripts différents et le souci de
garder la cohérence.
Fixes #2635
Historique
#1 Mis à jour par Joël Cuissinat il y a plus de 12 ans
+ débat ouvert dans :
http://dev-eole.ac-dijon.fr/issues/2614#note-4
#2 Mis à jour par Daniel Dehennin il y a environ 12 ans
- Version cible changé de Mises à jour 2.3.4 RC à Mises à jour 2.3.5 RC
C’est pas beau de hardcoded.
#3 Mis à jour par Emmanuel GARETTE il y a environ 12 ans
C’est pas beau de hardcoded.
Mettre un chemin dans une variable c'est aussi hardcoder le chemin.
On peut voir cette remarque de 2 façons :
- "ce n'est pas beau de dupliquer", la je suis d'accord ;
- "il faut laisser la possibilité à l'utilisateur de faire le point de montage ailleurs", la je suis moins d'accord.
Il me semble compliqué de ne pas hardcodé le point de montage (trop d'implication a différents endroit) pour un bénéfice (quasi) nul.
#4 Mis à jour par Benjamin Bohard il y a environ 12 ans
Si il y a consensus, on peut donc remplacer les '/mnt/sauvegardes' par MOUNT_DIR (déclarée en début de fichier)
#5 Mis à jour par Benjamin Bohard il y a environ 12 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit e65bc6ad1a34eccf11ef9fbd4d15e93932e6d47b.
#6 Mis à jour par Benjamin Bohard il y a presque 12 ans
- Statut changé de Résolu à Fermé
En attendant de pouvoir s'affranchir des valeurs "hardcoded"