[xfce4]Script de lancement starxfce4(abandonné)

[karhu]

Bonjour

À la lecture du script

startxfce4

j'ai l'impression d'avoir vu une ou deux améliorations possibles pour le test et l'initialisation des variables

$XDG_

À qui dois je m'adresser pour cela ?

Karhu.

[tuxce]

https://bugzilla.xfce.org/ ou http://www.xfce.org/community

[karhu]

Merci tuxce

Je me suis inscrit sur le forum pour soumettre mon idée. J'attends une réponse.
Dernière question, penses tu que je devrais proposer une correction plutôt que de suggérer une modification de leur part ?

Karhu.

[tuxce]

En règle générale, si on a une modification à apporter à un code, autant poster la modification.
Ceci dit, j'ai pas très bien saisi ce que tu proposes ... quelle est la correction ?

[karhu]

Par exemple :

Code : Tout sélectionner

if test "x$XDG_CONFIG_HOME" = "x"
then
  BASEDIR="$HOME/.config/xfce4/"
else
  BASEDIR="$XDG_CONFIG_HOME/xfce4"
fi

Si tu n'exporte pas $XDG_CONFIG_HOME (ce qui parait difficile au vu du code) elle reprendra sa valeur par défaut qui est : $HOME/.config.

Code : Tout sélectionner

if test "x$XDG_DATA_DIRS" = "x"
then
  if test "x/usr/share" = "x/usr/local/share" -o "x/usr/share" = "x/usr/share"; then
    XDG_DATA_DIRS="/usr/local/share:/usr/share"
  else
    XDG_DATA_DIRS="/usr/share:/usr/local/share:/usr/share"
  fi
else
  XDG_DATA_DIRS="$XDG_DATA_DIRS:/usr/share"
fi
export XDG_DATA_DIRS

Quel est l'intérêt de doubler "/usr/share" ?
Quel est le rôle de ce test et son intérêt ?

if test "x/usr/share" = "x/usr/local/share" -o "x/usr/share" = "x/usr/share"; then

Si tu es sous xfce4 regarde le script et donnes moi ton avis. Merci.

Karhu.

[tuxce]

Si tu n'exporte pas $XDG_CONFIG_HOME (ce qui parait difficile au vu du code) elle reprendra sa valeur par défaut qui est : $HOME/.config

Ben c'est le but (ou alors j'ai pas compris ce que tu veux dire)


En ce qui concerne les tests en double, en fait quand tu regardes un script ou programme etc., il faut plutôt regarder sa version avant installation:
http://git.xfce.org/xfce/xfce-utils/tre ... rtxfce4.in

Et dans ce fichier, tu vois qu'en fait ils testent:

Code : Tout sélectionner

"x@_datadir_@" = "x/usr/local/share" -o "x@_datadir_@" = "x/usr/share"

@_datadir_@ étant un motif qui est remplacé lors du "make"
Donc en fait ils vérifient si xfce-utils a bien été installé avec @_datadir_@ correspondant à /usr/share ou /usr/local/share

[karhu]

D'accord pour le make.
Seulement lorsque tu lance startxfce4 c'est le shell qui l'interprète, et dans les tests :

Code : Tout sélectionner

if test "x$XDG_CONFIG_HOME" = "x"
  if test "x/etc" = "x/etc"; then

"x" n'est pas remplacé, puisque ce n'est pas (plus) un motif, et le résultat sera toujours vrai dans le deuxième cas. D'où ma question de savoir s'il est utile de le garder dans le script pour le shell.
De même que

XDG_CONFIG_DIRS="/etc/xdg:/etc/xdg"

ne me parait pas pertinent.
De plus je me demande s'il ne serait pas judicieux de prévoir une initialisation de ces variables dans /etc/profile ou $HOME/.bash_profile, de les tester dans le script startxfce4 et de les réinitialiser en cas d'absence à leur valeur par défaut.
C'est juste une suggestion, de toute façon je vais le faire chez moi pour voir, mais si tu penses que le sujet ne mérite pas de suite je laisse aller.
Je n'ai pas l'habitude du developpement communautaire alors je tatonne un peu, et il y a tellement de projets que je ne sais pas quoi faire pour aider. Mon poste était en quelque sorte, comme le disaient les Beatles : "Just a little help for my friends".

Karhu.

[tuxce]

De même que
XDG_CONFIG_DIRS="/etc/xdg:/etc/xdg"
ne me parait pas pertinent.

Ca, c'est dans un "else" qui n'est jamais atteint parce justement "/etc/xdg" = "/etc/xdg" !

De plus je me demande s'il ne serait pas judicieux de prévoir une initialisation de ces variables dans /etc/profile ou $HOME/.bash_profile, de les tester dans le script startxfce4 et de les réinitialiser en cas d'absence à leur valeur par défaut.

C'est justement ce qui est fait. Le test:

Code : Tout sélectionner

if test "x$XDG_CONFIG_HOME" = "x"

par exemple, est justement là pour voir si la variable est déjà déclarée.

Il faut prendre du recul, voir le besoin et surtout bien interpréter le script.
Les devs d'xfce ont besoin d'un script fonctionnant sur une multitude de configurations, ce script doit prévoir l'absence des $XDG_*, prévoir si xfce a été installé dans d'autres répertoires que /etc, /usr ..

[FoolEcho]

[HorsSujet]

Mon poste était en quelque sorte, comme le disaient les Beatles : "Just a little help for my friends".

"With a little help from my friends". Ouais, Ringo !

[/HorsSujet]

[karhu]

J'ai juste adapté le titre pour illustrer mon propos, et comme ils sont dissouent je n'ai rien à craindre !

Je comprends ton point de vue, sans toutefois le partager complètement, et donc je vais clore ce poste. J'espère trouver un autre moyen de participer autrement qu'en utilisant.
Karhu.

[FoolEcho]

[HorsSujet]

J'ai juste adapté le titre pour illustrer mon propos, et comme ils sont dissouent je n'ai rien à craindre !

Quand on cite, on cite. On ne déforme pas. "I'm fixing a hole where the rain gets in and stops my mind from wandering "
[/HorsSujet]

J'espère trouver un autre moyen de participer autrement qu'en utilisant.

Y a toujours de la doc qui attend...