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'
Related issues
Associated revisions
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
History
#1 Updated by Joël Cuissinat almost 12 years ago
+ débat ouvert dans :
http://dev-eole.ac-dijon.fr/issues/2614#note-4
#2 Updated by Daniel Dehennin over 11 years ago
- Target version changed from Mises à jour 2.3.4 RC to Mises à jour 2.3.5 RC
C’est pas beau de hardcoded.
#3 Updated by Emmanuel GARETTE over 11 years ago
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 Updated by Benjamin Bohard over 11 years ago
Si il y a consensus, on peut donc remplacer les '/mnt/sauvegardes' par MOUNT_DIR (déclarée en début de fichier)
#5 Updated by Benjamin Bohard over 11 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
Appliqué par commit e65bc6ad1a34eccf11ef9fbd4d15e93932e6d47b.
#6 Updated by Benjamin Bohard over 11 years ago
- Status changed from Résolu to Fermé
En attendant de pouvoir s'affranchir des valeurs "hardcoded"