Sed + File not found : Problème de syntaxe ?

Version imprimable

Bonjour, j´ai un petit projet perso mais je bute avec sed.
J´ai le code suivant

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
#!/bin/sh
# Recherche ip actuel
check_ip="1.2.3.4"
check_new=`grep -oP 'NEW=\K.*' ip_current.txt`
 
#Creation fichier ip_current.txt si n´existe pas
if [ -e ip_current.txt ]
then
    touch ip_current.txt
fi
#check ips en ip_current.txt
if [ $check_ip  != $check_new ]; then
    echo "IP Differente"
    echo "NEW=$check_ip" > ip_current.txt
    echo "OLD=$check_new" >> ip_current.txt
    sed 's/NEW=$check_new/NEW=$check_ip/g' ip_current.txt
 
    #sed 's/NEW=$check_new/NEW=$check_ip/g' /volume1/@appstore/DNSServer/named/etc/zone/master/local
    sed 's/OLD=\K.*/$check_ip/g' ./volume1/@appstore/DNSServer/named/etc/zone/master/local
 
 
else
    echo "Pas de changement de IP"
fi

C´est le sed suivant qui pose probleme :

Code:

sed 's/OLD=\K.*/$check_ip/g' ./volume1/@appstore/DNSServer/named/etc/zone/master/local

Il ne trouve pas le fichier "local", donc je me demande comment placer cette commande pour effectuer l´action.
Merci

17/12/2021, 12h36
al1_24

Bonjour,

Ma première supposition serait que, comme tu utilises un chemin relatif (commençant par ./) pour accéder à ton fichier, tu ne te trouves pas là où tu penses être quand le script s'exécute.
Il serait plus judicieux d'indiquer un chemin absolu (partant de la racine du système)

Bizarre, mais je comprends pas sed. Je vais reprendre un exemple simple pour eviter la confusion.

Le code est le suivant
Code:

1 2 3 4 5 check_ip="1.2.3.4" check_old="192.168.1.111" echo "NEW=$check_ip" > ip_current.txt echo "OLD=$check_old" >> ip_current.txt sed 's/$check_old/$check_ip/g' '/volume1/@appstore/DNSServer/named/etc/zone/master/local'
Avec sed je dois devoir remplacer "check_ip="1.2.3.4" " par " check_old="192.168.1.111" ".
dans le fichier "/volume1/@appstore/DNSServer/named/etc/zone/master/local"
En lançant le script j´ai ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 $ORIGIN local. $TTL 86400 local. IN SOA ns.local. mail.local. ( 1 43200 180 1209600 10800 ) local. NS ns.local. ns.local. A 192.168.1.111
Il m´affiche ce que j´ai dans le fichier mais fait pas le remplacerment.

17/12/2021, 13h20
N_BaH
les variables shell ne sont pas développées entre apostrophes :[
Code:

1 2 3 4 5 $ var='foo bar baz' $ echo "$var" foo bar baz $ echo '$var' $var
donc, sed ne trouve de correspondance, et ne fait en conséquence pas de substitution.

J´ai fait des modification comme ceci :

Code:

1
2
3
4
5
check_ip='1.2.3.4'
check_old='192.168.1.111'
    echo '$check_ip' > ip_current.txt
    echo '$check_old' >> ip_current.txt
    sed 's/'$check_old'/'$check_ip'/g' /volume1/@appstore/DNSServer/named/etc/zone/master/local

Maintenant il m´affiche :

Code:

1
2
3
4
5
6
7
8
9
10
11
$ORIGIN local.
$TTL 86400
local. IN SOA ns.local. mail.local. (
        1
        43200
        180
        1209600
        10800
)
local.  NS      ns.local.
ns.local.       A       1.2.3.4

Mais lorsque je regarde dans le fichier les modifications ne sont pas prises en compte.

17/12/2021, 13h36
disedorgue

Bon après, le \K ne fonctionne pas en sed, dans une discussion précédente, la réponse était pour grep pas sed.
17/12/2021, 13h41
superthx

Citation:

Envoyé par disedorgue

Bon après, le \K ne fonctionne pas en sed, dans une discussion précédente, la réponse était pour grep pas sed.

Non mais le code precedent me va.
Ce que j´essai de faire et de changer dans le fichier local juste une ip par une autre.
17/12/2021, 14h38
jack-ft

Citation:

Envoyé par superthx

Mais lorsque je regarde dans le fichier les modifications ne sont pas prises en compte.

En standard (et en gros), sed lit le contenu d'un fichier, fait les modifications et écrit le résultat des modifications sur la sortie standard (stdout).

Si tu veux que sed modifie le contenu d'un fichier, il faut utiliser l'option -i (cf. man sed).

Attention, certaines versions de sed ont une option -i sans argument et d'autres avec argument obligatoire: -i <backup extension>.

Je continu mon petit chemin pour mon script. J´ai fais quelques modifs.
Maintanenant j´ai ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#!/bin/sh
# Busca la ip actual
if [[ ! -e ip_current.txt ]]; then
    touch ip_current.txt
fi
check_ip='curl -silent -L http://checkip.amazonaws.com|tail -n 1'
echo 'NEW=$check_ip' > ip_current.txt
check_new=`grep -oP 'NEW=\K.*' ip_current.txt`
check_old=`grep -oP 'OLD=\K.*' ip_current.txt`
if [ "$check_ip" != "$check_new" ]; then
	echo "IP Differente"
	echo 'NEW=$check_ip' > ip_current.txt
	echo "OLD=$check_old" >> ip_current.txt
	sed -i "s/$check_old/$check_ip/g" local
else
	echo "No hay cambio de IP"
fi

Mais il me renvoi cet erreur :

Code:

sed: -e expression #1, char 26: unknown option to `s'

20/12/2021, 09h22
disedorgue

:cristal:
20/12/2021, 12h34
N_BaH

$check_old est vide.
et il y a une erreur deux lignes plus haut.

Je reviens vers vous un peu plus tard je vais poser une autre question sur sed dans le forum. Je reviens avec la soluce un peu plus tard.
Merci

Edit j´ai enfinnnn reussi. Ci dessous le code :

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
27
#!/bin/sh
# Creation fichier ip_current.txt si n´existe pas
if [[ ! -e ip_current.txt ]]; then
    echo "First Run!"
    echo "Creating file ip_current.txt"
    touch ip_current.txt
    check_ip=`curl -silent -L http://checkip.amazonaws.com|tail -n 1`
    check_current_dns=`sed  -n 's/^ns.local.* //p' local`
    echo "NEW=$check_ip" > ip_current.txt
    echo "CURRENT=$check_current_dns" >> ip_current.txt
fi
 
#Charge les valeurs
check_ip=`curl -silent -L http://checkip.amazonaws.com|tail -n 1`
check_current_dns=`sed  -n 's/^ns.local.* //p' local`
 
#Si l´adress Ip est differente la remplace
if [ "$check_ip" != "$check_current_dns" ]; then
    echo "IP Differente"
    echo "OLD=$check_current_dns" > ip_current.txt
    echo "NEW=$check_ip" >> ip_current.txt
 
    #sed -i 's|$check_current_dns|$$check_ip|g' local
    sed -i "s/$check_current_dns/$check_ip/" local
else
    echo "No IP change"
fi

Si vous avez des suggestions ou des améliorations je suis preneur.

20/12/2021, 16h42
N_BaH

ne pas utiliser [[ avec /bin/sh, parce que cette instruction n'est pas POSIX.
utiliser des chemins absolus.
pourquoi faire deux fois les checkIP quand le fichier current n'existe pas ?
20/12/2021, 18h47
superthx

Citation:

Envoyé par N_BaH

pourquoi faire deux fois les checkIP quand le fichier current n'existe pas ?

Le premier if c'est pour creer le fichier et les valeurs.
Le second if c'est pour les changements.
20/12/2021, 19h09
N_BaH

mais puisque de toutes façons les mêmes opérations sont réalisées, il ne faut pas les faire deux fois.

tu les fais une fois, et si le fichier n'existe pas, alors tu y envoies les données.
20/12/2021, 20h50
N_BaH

plus je tripote le code, plus je me dis que current.txt ne sert à rien, puisque l'adresse courante est entretenue et récupérée dans le fichier dns "local",
et la nouvelle adresse est, elle, récupérée sur chekIP.