Projet

Général

Profil

Tâche #29222

Scénario #18447: Hâpy doit permettre d’initialiser plusieurs images ISO et plusieurs tailles de disques vierges

Le script 99-init-hapy-master doit supporter « set -e »

Ajouté par Daniel Dehennin il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
30/10/2019
Echéance:
% réalisé:

100%

Temps estimé:
1.00 h
Restant à faire (heures):
0.0

Description

Le script utilise abondamment la formulation [ ${?} -ne 0] && echo something qui fait planter le script s’il est exécuter avec set -e et que ${?} n’est pas égal à 0.

Il est préférable d’utiliser la formulation [ ${?} -eq 0 ] || echo something, dans quel cas l’instruction renvoie toujours true, soit parceque ${?} vaut 0, soit par l’exécution de echo.

Révisions associées

Révision afd0f59b (diff)
Ajouté par Daniel Dehennin il y a plus de 4 ans

Init: better support of shell “set -e”

Under shell “set -e” we must not use “[ ! command1 ] && command2” since
if the first command fails the instruction line fails and the script
exit.

It's more robust and simple to use “[ command ] || command2”.

  • postservice/99-init-hapy-master: replace all “&&” linked instruction
    by “||” equivalent.

Ref: #29222

Révision 70049997 (diff)
Ajouté par Daniel Dehennin il y a plus de 4 ans

Init: fix cosmetic alignment and syntax

  • postservice/99-init-hapy-master (addISOImage): do not return 0 after
    an “echo”.

Ref: #29222

Historique

#1 Mis à jour par Daniel Dehennin il y a plus de 4 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Daniel Dehennin

#2 Mis à jour par Daniel Dehennin il y a plus de 4 ans

  • Statut changé de En cours à Résolu
  • % réalisé changé de 0 à 100
  • Restant à faire (heures) changé de 1.0 à 0.0

#3 Mis à jour par Daniel Dehennin il y a plus de 4 ans

Pas d’erreur avec le paquet eole-one-master 2.7.2-19 :

run-parts: executing /usr/share/eole/postservice/99-init-hapy-master instance
***** Création de DSK-40-Go *****
ID: 0

***** Création de DSK-50-Go *****
ID: 1

***** Création de DSK-100-Go *****
ID: 2

***** Création de l'image eole-2.7.2-a0-alternate-amd64.iso *****

ID: 3
       Création terminée, l'image a le statut : rdy

***** Création de install-eole-2.7.2-a0-amd64 *****
ID: 0

***** Création de l'image eole-2.7.1.1-alternate-amd64.iso *****

ID: 4
       Création terminée, l'image a le statut : rdy

***** Création de install-eole-2.7.1.1-amd64 *****
ID: 1

***** Création de l'image eole-2.6.2.2-alternate-amd64.iso *****

ID: 5
       Création terminée, l'image a le statut : rdy

***** Création de install-eole-2.6.2.2-amd64 *****
ID: 2

#4 Mis à jour par Daniel Dehennin il y a plus de 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF