[1: command not found

Version imprimable

16/02/2023, 14h37
videk

2 pièce(s) jointe(s)

[1: command not found

Bonjour

avez-vous une idée sur le pb

Pièce jointe 634351

Pièce jointe 634352
16/02/2023, 16h16
N_BaH

modération
- le titre de ton sujet doit être explicite.
- il est préférable, quand on poste du code, de ne pas envoyer des captures d'écrans, mais autant de copier/coller.
- en cherchant sur le net avant de poster (c'est demandé dans la charte du forum), tu aurais trouvé une solution au problème : il faut des espaces autour des crochets.
16/02/2023, 16h21
N_BaH

aux crochets, je préfère test qui dit bien ce qu'il fait, et "désolidarise" la commande du if, évitant ainsi toute confusion.

cd ~/quelquePart, donc tu sais où tu te déplaces, tu sais où tu es : y=`pwd` ne sert alors à rien.
d'une manière générale, dans un script, cd n'est pas utile.

test -f fichier, et ... ?
ah, oui ! a=$? :roll:
pourquoi pas tout de suite if test -f fichier (comme je le disais au début de mon message) ? :)
16/02/2023, 16h41
Sve@r
Citation:

Envoyé par videk

avez-vous une idée sur le pb

Il faut espacer les crochets de l'évaluation => if [ $a -eq 0 ] et non if [$a -eq 0]. Ou alors, comme le dit N_Bah, utiliser test qui est la commande d'origine => if test $a -eq 0.

A partir de là, ton script fonctionnera. Mais question finesse...:?
Aux remarques de N_BaH, qui bien évidemment gardent toutes leur pertinences, on peut rajouter celles-ci
- on ne teste pas si un état vaut 1 mais s'il est différent de 0 (parce qu'il peut valoir toute la gamme de 1 à 255, chaque valeur permettant d'indiquer un souci X ou Y)
- plutôt que de tester s'il vaut 1, puis re-tester s'il vaut 0, on testera plus intelligemment s'il est différent de 0 et sinon, c'est que le fichier existe (un seul test au lieu de deux)
- si tu rajoutes une instruction (style echo) entre le test et la récupération de l'état, tu récupères l'état de la dernière instruction et non plus celui du test. Donc il vaut mieux regrouper la commande et son état en une ligne => test -f resultat; status=$?
Citation:

Envoyé par N_BaH

aux crochets, je préfère test qui dit bien ce qu'il fait

Mince, hier (oui oui, j'ai vraiment fait ça hier, à prendre au sens littéral) j'ai passé tous mes scripts et ai remplacé les "test" par des crochets.
Mais voici pourquoi
- j'ai remarqué que les scripts système (/etc/profile, .profile par défaut) utilisaient les crochets
- etant donné qu'il est sorti les doubles crochets [[ ... ]] permettant des tests avancés, cette syntaxe n'ayant pas d'équivalent littéral (enfin je crois), je me suis dit que désormais la syntaxe "crochets" devenait une syntaxe à plébisciter, permettant ainsi de changer plus facilement [ ... ] en [[ ... ]] et inversement si besoin (d'ailleurs justement la semaine dernière j'ai utilisé pour la première fois les doubles crochets afin de regarder si "$TERM" débutait par "xterm" => [[ $TERM = xterm* ]]
Mais bon, ça reste aus goûts de chacun. 8-)
16/02/2023, 16h52
videk

1 pièce(s) jointe(s)

Merci N_BaH

pour info, je suis débutant

voilà j'ai rectifié le script aussi, j'ai tapé la commande sur le terminal avant
mais j'ai toujours un problème

Pièce jointe 634358
16/02/2023, 16h59
Sve@r

Citation:

Envoyé par videk

Merci N_BaH

Ouais, de rien, ça fait toujours plaisir !!!

Citation:

Envoyé par videk

pour info, je suis débutant

Oh? Sérieux??? On s'en serait jamais douté !!!

Citation:

Envoyé par videk

voilà j'ai rectifié le script aussi, j'ai tapé la commande sur le terminal avant
mais j'ai toujours un problème

Parce qu'être débutant t'empêche de lire et de recopier ce que N_BaH a écrit avec exemple à la clef???
16/02/2023, 17h09
videk

1 pièce(s) jointe(s)
Citation:
Envoyé par Sve@r

Il faut espacer les crochets de l'évaluation => if [ $a -eq 0 ] et non if [$a -eq 0]. Ou alors, comme le dit N_Bah, utiliser test qui est la commande d'origine => if test $a -eq 0.

A partir de là, ton script fonctionnera. Mais question finesse...:?
Aux remarques de N_BaH, qui bien évidemment gardent toutes leur pertinences, on peut rajouter celles-ci
on ne teste pas si un état vaut 1 mais s'il est différent de 0 (parce qu'il peut valoir toute la gamme de 1 à 255, chaque valeur permettant d'indiquer un souci X ou Y)
plutôt que de tester s'il vaut 1, puis re-tester s'il vaut 0, on testera plus intelligemment s'il est différent de 0 et sinon, c'est que le fichier existe (un seul test au lieu de deux)
si tu rajoutes une instruction (style echo) entre le test et la récupération de l'état, tu récupères l'état de la dernière instruction et non plus celui du test. Donc il vaut mieux regrouper la commande et son état en une ligne => test -f resultat; status=$?

Mince, hier (oui oui, j'ai vraiment fait ça hier, à prendre au sens littéral) j'ai passé tous mes scripts et ai remplacé les "test" par des crochets.
Mais voici pourquoi
j'ai remarqué que les scripts système (/etc/profile, .profile par défaut) utilisaient les crochets
etant donné qu'il est sorti les doubles crochets [[ ... ]] permettant des tests avancés, cette syntaxe n'ayant pas d'équivalent littéral (enfin je crois), je me suis dit que désormais la syntaxe "crochets" devenait une syntaxe à plébisciter, permettant ainsi de changer plus facilement [ ... ] en [[ ... ]] et inversement si besoin (d'ailleurs justement la semaine dernière j'ai utilisé pour la première fois les doubles crochets afin de regarder si "$TERM" débutait par "xterm" => [[ $TERM = xterm* ]]

Mais bon, ça reste au goûts de chacun.
ok ca marche merci le pb etait l'espace à la condition if

Pièce jointe 634359

test -f ne teste pas l'existence, mais le type. Donc si "resultat" est un lien symbolique, un pipe, un fichier spécial ou un dossier, le test renvoie faux bien qu'il existe. Pour tester l'existence pure c'est test -e
Code:

1 2 3 #!/usr/bin/bash test -e "resultat" && echo "Le fichier resultat existe" || echo "Le fichier resultat n'existe pas" [ -e "resultat" ] && echo "Le fichier resultat existe" || echo "Le fichier resultat n'existe pas"
Pour les chaines, il vaut mieux en général utiliser les doubles quotes plutôt que les quotes simples car les doubles quotes conservent certaines propriétés intéressantes
Code:

1 2 3 #!/usr/bin/bash echo 'Je me trouve dans $(pwd)' echo "Je me trouve dans $(pwd)"
Les simples quotes ont leur utilité aussi (par exemple quand justement on ne veut pas transformer le texte) mais pas ici.

Citation:
Envoyé par Sve@r
test -f ne teste pas l'existence, mais le type. Donc si "resultat" est un lien symbolique, un pipe, un fichier spécial ou un dossier, le test renvoie faux bien qu'il existe. Pour tester l'existence pure c'est test -e
Code:

1 2 3 #!/usr/bin/bash test -e "resultat" && echo "Le fichier resultat existe" || echo "Le fichier resultat n'existe pas" [ -e "resultat" ] && echo "Le fichier resultat existe" || echo "Le fichier resultat n'existe pas"
oui, je sais sauf que je veux fair des essais des commandes suivants dans un script
donc le message n'explique pas la commande

test -e <nom_fichier> : vrai si l’argument existe
test -f <nom_fichier> : vrai si l’argument est un fichier ordinaire
test -d <nom_fichier> : vrai si l’argument est un répertoire
test -L <nom_fichier> : vrai si l’argument est un lien symbolique
test -r <nom_fichier> : vrai si on a le droit de lire le fichier
test -x <nom_ficher> : vrai si on a le droit d’exécuter le fichier
test -w <nom_ficher> : vrai si on a le droit d’écrire dans le fichier
test -s <nom_fichier> : vrai si l’argument existe et non vide

16/02/2023, 18h28
N_BaH
Citation:

test -e <nom_fichier> : vrai si l’argument existe

non.
Code:

1 2 $ help test | grep -- -e\\\> -e FICHIER Vrai si le fichier existe.
la vérification de l'existence d'un argument se fait par -n, parce qu'un argument (ou "paramètre") est forcément une chaîne.

il faut entendre "fichier" au sens large, puisque sur Linux tout est fichier (fichier, lien, répertoire, périphérique...)
Code:

1 2 3 $ test -e /dev/sda1; echo $? 0 $
la première partition du premier disque dur du système est un bien fichier.
01/03/2023, 19h04
Jipété
Bonsoir,

un peu HS mais toujours utile :
Citation:
Envoyé par N_BaH

Code:

1 2 3 $ test -e /dev/sda1; echo $? 0 $

la première partition du premier disque dur du système est un bien fichier.
On n'avait pas dit qu'il ne fallait plus utiliser cette manière d'écrire, car au cas où plusieurs disques, /dev/sda1 n'est pas toujours le premier disque et autant prendre de bonnes habitudes dès le début, non ? :P

Citation:

Les noms /dev/sd* ne sont pas persistants. Ils sont attribués dans l’ordre de détection des disques. Le disque de sauvegarde pourrait très bien être /dev/sda s’il est présent au démarrage et détecté avant le disque interne, ou /dev/sdc si un autre disque ou clé USB est présent et vu comme sdb…
Avec un identifiant persistant, ce n’est pas un problème. La syntaxe est UUID=xxxx ou LABEL=xxxx (infos affichées par blkid) au lieu de /dev/sdb1.

source : https://www.debian-fr.org/t/besoin-d...egarde/87902/5

avec bien sûr la blagounette suivante, trouvée dans man blkid : plutôt utiliser lsblk.
:coucou:
02/03/2023, 17h14
jack-ft
Citation:

Envoyé par Sve@r

Donc il vaut mieux regrouper la commande et son état en une ligne => test -f resultat; status=$?[/list]

Sur le principe, je suis d'accord... :ccool:

Attention ! Avec le "zsh", la variable status est "spéciale/réservée/protégée/read-only" (cochez la case qui vous convient)... :aie:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 % cat test.zsh #!/bin/zsh foo() { local status echo 'coucou'; status=$? echo "status: $status" } foo % ./test.zsh coucou foo:2: read-only variable: status
Quand j'ai voulu porter mes scripts sh ou bash en zsh, j'ai dû renommer les variables status en statut, par exemple !

Et il me semble que $status a la même valeur que $?.
02/03/2023, 17h26
Sve@r

Citation:

Envoyé par jack-ft

Attention ! Avec le "zsh", la variable status est "spéciale/réservée/protégée/read-only" (cochez la case qui vous convient)... :aie:

Ah oui !!! C'est pour ça que "vi" me met la variable en rouge. Je sentais bien un truc dans ce genre mais je cherchais dans bash et non dans zsh...

Merci de cette info :ccool: