marionnet-dev team mailing list archive

[Jean-Vincent Loddo <Jean-vincent.Loddo@xxxxxxxxxxxxxxxxxxxx>]

Bonjour,

Finalement nous devrions avoir le beurre et l'argent du beurre. La 
variable MARIONNET_TMPDIR peut être définie par l'administrateur si 
vraiment nécessaire (mais normalement il n'y a pas besoin). S'il le 
fait, cette valeur est prioritaire par rapport à TMPDIR, mais c'est 
certainement en connaissance de cause... (TMPDIR est plus 
"general-purpose" par rapport à MARIONNET_TMPDIR).

Toutefois, rien n'est définitif, si un utilisateur veut changer ce 
réglage, il peut le faire par un simple :

MARIONNET_TMPDIR=blabla marionnet

J'ai fait les modifs qu'il fallait dans ce sens.
Merci encore pour l'idée et le patch,
J.V.

Le 14/09/2012 08:56, Pierre BLONDEAU ( sysadmin dept-info ) a écrit :
> 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 ;-))