Discussion :

[Sve@r]

Bien le bonjour

J'étais en train de bosser sur grub pour mieux comprendre certains détails, et suis arrivé dans le fichier "/etc/grub.d/05_debian_theme"

Et là je vois ces deux lignes qui me surprennent un peu (23 et 24 chez moi)

Code bash :

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

1
2
# We want to work in /boot/grub/ only.
test -d /boot/grub; cd /boot/grub

Le commentaire est explicite sur le but... mais le code n'y correspond pas. Si je ne me trompe pas, le test est fait pour rien. Si /boot/grub n'existe pas ou n'est pas répertoire ben de toute façon rien. Le cd sera quand-même bel et bien fait, renverra une erreur mais la suite du script continuera à s'exécuter.

Quelqu'un aurait-il un avis?

Merci à tous

[N_BaH]

énorme.
«
le petit bonhomme du passage piéton est-il rouge ? je traverse.
»

n'y aurait-il pas un set -e au début du fichier ?

edit: si, il y a bien un set -e :

Code :

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

1
2
3
4
5
#!/bin/bash
 
set -e
 
test -d "$1"; echo ok

[Sve@r]

n'y aurait-il pas un set -e au début du fichier ?

edit: si, il y a bien un set -e

Ah bien vu

énorme.

Effectivement
Toutefois, soyons honnête, c'est là la différence entre quelqu'un qui fait du shell H/24, qui pense shell H/24 (et qui, par conséquent, a les réflexes super affûtés), et quelqu'un qui utilise le shell pour ses travaux d'intérêt général et qui, tout en sachant pas mal de choses en shell, n'y pense pas immédiatement de façon automatique.

Un grand merci quand-même pour m'avoir rafraichi mes connaissances

[N_BaH]

énorme

je parlais du code, pas de ta question.

[Flodelarab]

Bonjour

Intéressant.

À noter :

• Le code retour est le code retour de la commande qui a planté.
• On parle des commandes qui renvoient un code retour dans le vent. Une commande dans la condition d'un "if" peut renvoyer faux sans faire arrêter le script.

[hizoka]

Je ne connaissais pas ce set -e... violent...

[jack-ft]

Violent, mais efficace.

L'idée, c'est de produire du code qui prend en compte toutes les erreurs potentielles... un peu comme dans les langages de programmation "classiques".