← Back to team overview

marionnet-dev team mailing list archive

Re: Patch pour l'utilisation des tmpfs et variable dans TMPDIR

 

Bonjour,

Pour l'ordre, il y a du pour et du contre dans les deux cas.

Pour $MARIONNET_TMPDIR, $TMPDIR, etc... on part du principe que l'administrateur à forcement raison ( ce qui est vrai ! ;) ) et qu'il sais ce qu'il fait ( ce qui est vrai aussi ! ), ça peut éviter le problème avec certain étudiant.

Pour $TMPDIR, $MARIONNET_TMPDIR, etc... C'est le fonctionnement par défaut de tous les logiciels sous linux. si l'administrateur a juste installé marionnet sans configurer MARIONNET_TMPDIR, l'utilisateur ne pourra pas le change car la variable qu'il pourrait éventuellement définir sera ignorer.

Par défaut la variable TMPDIR n'est pas défini, c'est vraiment une action de l'utilisateur. Donc tant pis pour lui si il là définit avec un mauvais path.

Personnellement, je préfère le choix numéro 2 car c'est le comportement standard.

Il y a également une troisième possibilité, des logiciels comme firefox proposent de définir des variables "fixes". C'est à dire que si elle sont dans le fichier de configuration est caractérisées comme fixes, l'utilisateur ne pourra pas les modifier sinon il le pourra. mais ça commence à faire compliquer.

A+

PS : pour les patchs, je travail en étroite collaboration avec Davy, donc il a bien participé aux deux patchs que nous avons proposé. Et si c'est lui qui en propose un il y aura de grande chance que j'y participe ;)

On 13/09/2012 22:30, Jean-vincent.Loddo@xxxxxxxxxxxxxxxxxxxx wrote:
Bonsoir,

(je remets la liste en CC car la discussion me semble pertinente et
intéressante)

On Thu, 13 Sep 2012 19:52:49 +0200, Davy Gigan wrote:
Bonsoir,

Le 13/09/2012 17:16, Jean-Vincent Loddo a écrit :> $MARIONNET_TMPDIR
$TMPDIR
/tmp
$PWD
~/tmp
~/

Est-ce qu'il y a des arguments contre ce choix ?

Si une variable spécifique à marionnet est introduite, il semble logique
en effet d'utiliser cet ordre.

Ok, on est bien d'accord.

Cependant, si marionnet gère correctement l'existence de la variable
d'environnement TMPDIR, il n'y a pas vraiment de raison d'introduire une
nouvelle variable.
En effet, un utilisateur Unix averti saura faire dans son shell :
  TMPDIR=/chemin marionnet

Oui, TMPDIR est gérée correctement : actuellement, si elle est définie,
elle a la priorité. Ceci dit, le patch proposé avec la nouvelle variable
MARIONNET_TMPDIR n'est pas sans intérêt, si on pense aux spécificités
(taille et support des sparse) du répertoire associé à Marionnet par
rapport à un répertoire temporaire *ordinaire*. Ce dernier sera utilisé
tout naturellement par l'administrateur du système pour affecter TMPDIR,
sans pour autant convenir parfaitement à Marionnet.

Donc, pour résumer, je vais prendre le patch en le modifiant pour avoir
la priorité ci-dessus, sur laquelle nous sommes tous d'accord.

Au pire, quitte à patcher le logiciel, il est peut-être possible de
simplement surcharger la variable TMPDIR directement dans le
marionnet.conf au lieu d'introduire une nouvelle variable ?

Oui mais ça revient au même, donc autant avoir une nouvelle variable. La
seule différence concernerait les processus lancés par Marionnet qui
auraient une valeur plus appropriée pour TMPDIR. Il me semble,
considérée la nature des processus lancés, que ça ne changerait rien.

A+
Jean-Vincent

PS. Dans le dernier message de commit du trunk, j'ai remercié une
personne, Pierre, qui m'avait envoyé le patch. Tu es peut-être co-auteur
du patch ? (si c'est le cas à l'avenir, précisez-moi quand il y a
plusieurs auteurs pour que je puisse remercier tout le monde lors du
commit, même si vous y tenez pas, moi j'y tiens ;-))



--
----------------------------------------------
Pierre BLONDEAU
Administrateur Systèmes & réseaux
Université de Caen
Laboratoire GREYC, Département d'informatique

tel	: 02 31 56 75 42
bureau	: Campus 2, Science 3, 406
----------------------------------------------


Follow ups

References