Project

General

Profile

Anomalie #2635

Chemins "en dur" dans posttemplate/00-bacula

Added by Joël Cuissinat almost 8 years ago. Updated over 7 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Category:
-
Start date:
12/21/2011
Due date:
% Done:

100%

Estimated time:
0.25 h
Distribution:
EOLE 2.3

Description

Il y a 2 occurrences de /mnt/sauvegardes alors que juste au-dessus :

MOUNT_DIR='/mnt/sauvegardes'


Related issues

Follows eole-bacula - Anomalie #2407: Erreur provoqué par chown sur le montage /mnt/sauvegardes (non démonté) Fermé 11/16/2011

Associated revisions

Revision e65bc6ad (diff)
Added by Benjamin Bohard over 7 years ago

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

#2 Updated by Daniel Dehennin over 7 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 7 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 7 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 7 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#6 Updated by Benjamin Bohard over 7 years ago

  • Status changed from Résolu to Fermé

En attendant de pouvoir s'affranchir des valeurs "hardcoded"

Also available in: Atom PDF