Discussion :

[ctac_]

Bonjour,
J'ai un menu contextuel dans xfce/thunar qui me permet d'éditer un fichier texte avec vi dans une fenêtre xterm.
Je m'en sert principalement pour modifier les fichiers systèmes.
Pour ceux qui voudraient tester et me proposer une solution ...
J'ai ajouté dans ~/.config/Thunar/uca.xml

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
<action>
	<icon>gvim</icon>
	<name>Modifier avec Vi</name>
	<unique-id>Menu_Perso-1</unique-id>
	<command>~/bin/Open_with_vi %f</command>
	<description>vi editor</description>
	<patterns>*</patterns>
	<other-files/>
	<text-files/>
</action>

J"ai ajouté dans ~/.config/Thunar/accels.scm

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

; (gtk_accel_path "<Actions>/ThunarActions/uca-action-Menu_Perso-1" "")

Mon fichier exécutable ~/bin/Open_with_vi

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#!/bin/sh
file="${1##*/}"
cmd="vi"
test $( ls -l "$1" | awk '{ print $3 }' ) != "$USER" && 
{
  comp="'"
  cmd="echo \"$file\" is not your file so you must set to root: ; su -l root -c '$cmd -c \"cd ${1%/*}\"" 
  xterm -title 'Vi Editor of '"$file" \
        -fa 'Monospace Regular' \
        -fs 15 \
        -e "$cmd '\''$1'\''$comp || \
           {
             tput clear
             echo Not authorized to edit \"$file\"
             echo Sorry
             sleep 3
           }"
} ||
xterm -title 'Vi Editor of '"$file" \
      -fa 'Monospace Regular' \
      -fs 15 \
      -e "$cmd '$1'"

Ici; j'ai 2 commandes xterm.
Je cherche a en avoir qu'une seule qui s'applique a tous les cas.
Mais là; je séche.
Actuellement, ça fonctionne sur debian, OpenBSD et NetBSD.
Je ne cherche pas une solution bash.
Si une petite suggestion vous vient a l'esprit, n'hesitez pas.
N_BaH va faire des bonds avec l'utilisation de ls mais s'il y a une meilleure solution ...

Petite explcation du script:

Si le fichier m'appartient, pas de probléme, vi est lancé.
Si c'est un fichier système, on demande le mot de passe root avant de lancer vi.
La commande

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

su -l

change le répertoire courrant vers /root.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

su -l root -c '$cmd -c \"cd ${1%/*}\"

permet de positionner vi dans le répertoire du fichier.
A mon avis trés utile pour faire une copie de sauvegarde.
Il reste encore a traiter le cas du fichier qui n'appartient ni a root, ni a moi mais on verra plus tard.
Cordialement.

[CosmoKnacki]

N_BaH va faire des bonds avec l'utilisation de ls mais s'il y a une meilleure solution ...

Pour être plus précis, N_BaH est l'un des rares qui a encore la volonté de réagir quand il voit ça. C'est pas la mer à boire de se pencher sur le man de find afin de contrôler la sortie.

Sinon désolé mais je n'utilise ni xfce, ni thunar... Pour une prochaine peut-être.

[Flodelarab]

Bonjour

Les 2 alternatives à "ls" sont "find" et "stat". Ici, c'est stat.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

stat -c "%U" *

En ce qui concerne l'utilisation d'une seule commande xterm, pourquoi ne pas mettre en facteur commun ?
Tu as des alternatives, mais elles finissent toutes par lancer xterm.
Donc, en affectant les variables dans les "faux-if", il ne reste plus qu'à finir par un seul xterm général.
Était-ce ta question ?

[ctac_]

Bonjour et merci du retour.

Je m'escrimais a vouloir placer le nom du fichier dans l'option -e de xterm.
En créant la commande compléte dans la variable cmd, ça va tout de suite mieux.

@Flodelarab : C'est vrai que stat serait parfais mais les options changent.
Le -c sur debian devient -t sur OpenBSD.

@CosmoKnacki : En grattant un peu, j'ai bien trouvé une alternative avec find surtout que je n'ai pas besoin de récupérer le propriétaire du fichier.

En effet, vi est sympa, même sous root, il ne change ni le propiétaire ni le group.
Il n'y a que la copie de sauvegarde avec :w 'Nom de la sauvegarde' qui appartient a root.

Voici le script :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
#!/bin/sh
file="${1##*/}"
cmd="vi"
rep="${1%/*}"
if test "$(find "$rep" -maxdepth 1 -name "$file" ! -user "$USER")" ; then
  cmd="echo \"$file\" is not your file so you must set to root: ; \
    su -l root -c '$cmd -c \"cd $rep\" '\''$1'\''' || \
    {
      tput clear
      echo Not authorized to edit \"$file\"
      echo Sorry
      sleep 3
    }"
else
  cmd="$cmd '$1'"
fi
xterm -title 'Vi Editor of '"$file" \
      -fa 'Monospace Regular' \
      -fs 15 \
      -e "$cmd"

Cordialement.

[N_BaH]

Bonjour,

ne suffirait-il pas de tester que le fichier est "inscriptible" (writable) par l'utilisateur en cours ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if test -w "$chemin/$fichier"

ls ...

[ctac_]

Bonjour,
Effectivement, nettement plus simple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if test -w "$chemin/$fichier"

Il me semblait bien aussi en parlant de ls

J'ai aussi modifié le cd (dans le shell plutôt que dans vi)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

su -l root -c 'cd $rep ; $cmd ...

a la place de :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

su -l root -c '$cmd -c \"cd $rep\" ...

Merci.
Cordialement.

[N_BaH]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
K I S S !
e t i t
e   m u
p   p p
    l i
    e d