Mise à jour globale de l'outil

[ayoahha]

Quelques questions en suspend :

1. Les problèmes des cd <dir>, sans traiter les erreurs Donc plutôt cd <dir> || exit OU une fonction pour bien quitter le programme si les change dir échouent !

2. Dans secureboot.sh MODULES_DIR et KBUILD_DIR n'ont pas l'air d'être utilisés... A voir

[Xdavius]

Les " " ne sont pas obligatoires en bash sur les variables (mais le sont avec sh je crois)

Pour les 2 variables effectivement elles ne sont actuellement plus utilisés (ca date d'une ancienne version de dev. pour l'instant je laisse actif car je dois ajouter des trucs au fichier.

je laisse ouvert le commit faut que je regarde tout en détail .

merci !

[ayoahha]

Les " " ne sont pas obligatoires en bash sur les variables (mais le sont avec sh je crois)

Pour les 2 variables effectivement elles ne sont actuellement plus utilisés (ca date d'une ancienne version de dev. pour l'instant je laisse actif car je dois ajouter des trucs au fichier.

je laisse ouvert le commit faut que je regarde tout en détail .

merci !

Non les " sont des bonnes pratiques de coding que ce soit SH ou BASH, on évite le globbing. D'ailleurs tu les mets des fois et d'autres non

OK pour les 2 variables pour ça que j'y ai pas touché !

Reste le coup des CD sans gestion d'erreur...

[Xdavius]

Les " " ne sont pas obligatoires en bash sur les variables (mais le sont avec sh je crois) Pour les 2 variables effectivement elles ne sont actuellement plus utilisés (ca date d'une ancienne version de dev. pour l'instant je laisse actif car je dois ajouter des trucs au fichier. je laisse ouvert le commit faut que je regarde tout en détail . merci !

Non les " sont des bonnes pratiques de coding que ce soit SH ou BASH, on évite le globbing. D'ailleurs tu les mets des fois et d'autres non

Du coup faut que je regarde où je les mets et où je les mets pas. le globbing je sais pas ce que c'est ^^

OK pour les 2 variables pour ça que j'y ai pas touché !

oui on laisse comme ça

Reste le coup des CD sans gestion d'erreur...

c'est a dire ?

J'ai regardé les fautes de frappe XD je m'y colle rapidement.

[ayoahha]

Du coup faut que je regarde où je les mets et où je les mets pas. le globbing je sais pas ce que c'est ^^

Alors mettre des " (comme ce que tu as fait sur certaines variables, mais pas toutes) est une bonne pratique quand par exemple dans le string tu as des espaces, on évite ainsi les erreurs inattendues. Ca peut aussi empêcher l'interprétation ou la mauvaise interprétation des caractères type * ou $ par ex.

• C'est aussi une bonne pratique et une sécurité pour éviter les injections de code. Exemple avec une chaine ";rm -rf /" je te laisse en déduire ce qu'il se passerait si tu entoures pas tes variables de " pour ne pas l'interpréter ;)

Édit : je t'ai fait toutes les modifs dans les commits donc pas besoin de rechercher

Reste le coup des CD sans gestion d'erreur... c'est a dire ?

Généralement, pour éviter des soucis de dossiers inexistants, on gère la sortie du script sur les Change Directory Par ex. :

• cd "$mon_dossier" || exit
• ou peut être mieux [ -d "$mon_dossier" ] && cd "$mon_dossier" || echo "Le répertoire cible n'existe pas."
• ou [ -d "$mon_dossier" ] && cd "$mon_dossier" || echo "Le répertoire cible n'existe pas."; mkdir -p "$mon_dossier" Toujours pour faire bonne mesure

[Xdavius]

Je close cette pull request.