Télécharger les sources d'OpenBSD

Version imprimable

Bonjour,

Je viens de retrouver un script suite à une perte de données :mouarf:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
get_base() {
 check_usergroup
 if [ $? -eq 0 ]; then
    while true; do
      echo -n "Get base system? (y/n) [y] "
      read answer
      case $answer in
      ""|y|Y)
         if (cd /usr && cvs checkout -rOPENBSD_6_6 -P src); then
             echo ">> Base system fetched."
         else
                 echo ">> Unable to fetch the base system!"
             fi
         return 0
         ;;
         n|N)
         echo "GoodBye!"
         break
         ;;
           *)
         echo "$answer: is not allowed!"
         ;;
       esac
   done
 fi
}

Je me demande si le test en bleue est toujours utile ?

En vous remerciant de votre réponse.

22/05/2020, 17h48
Flodelarab

Bonjour :coucou:

Citation:

check_usergroup
if [ $? -eq 0 ]; then

Moi, je m'demande surtout pourquoi faire ça en 2 lignes.

Code:

if check_usergroup; then
23/05/2020, 01h05
jack-ft
Citation:
Envoyé par Olivier Regnier

Je viens de retrouver un script suite à une perte de données :mouarf:

Code:

1 2 3 4 5 6 7 8 9 get_base() { check_usergroup if [ $? -eq 0 ]; then while true; do echo -n "Get base system? (y/n) [y] " [snip] done fi }

Je me demande si le test en bleue est toujours utile ?
Je suis d'accord avec la remarque de Flodelarab.

Et, manifestement, l'auteur de la fonction get_base a voulu que la récupération interactive de l'openBsd ne soit proposée qu'après vérification d'un retour sans anomalie de l'appel à check_usergroup...
C'est du moins ce que dit le code (plus que les commentaires du code).
24/05/2020, 19h27
ctac_

Bonjour,
Je suis d'accord avec Flodelarab.
Il serait de plus intéressant de voir la fonction check_usergroup.
Cordialement.