For Schleife in eine Variable(String) einlesen

[JCB]

Moin,

Folgendes Script soll checken ob es auf dem Server neue Dateien eines bestimmten Musters gibt und diese dann herunterladen. Soweit ist mein Script schon fertig, nur habe ich aus VB Zeiten noch die Zeile in erinnerung die eine For-Schleife in einen String einlesen kann. Die Bash sträubt sich da noch etwas. Was mache ich da falsch?? Nicht lachen, bin noch nicht der Mega Coder, aber bis auf die Schleife macht es was ich will Der String soll dann alle Dateinamen die downzuloaden sind, getrennt mit Leerzeichen enthalten.

#!/bin/sh
cd /pub/dnb/ftp/gb/fromgb

#liste holen und in lokaler Datei speichern
lftp -c "open -u username,password ftp.server.com/gets; ls C???M*.zip > /var/log/globalbatch"

#formatieren zu datei namens gbcurrent
cut -f 12 -d " " /var/log/globalbatch > /var/log/gbcurrent

#gbload, das was letztendlich runtergeladen werden soll als Liste erstellen
comm -3 /var/log/gbcurrent /var/log/gbold > /var/log/gbload

#gbload in variable einlesen

for i in `cat /var/log/gbload`; do

    downloads=$downloads$i;
done;


lftp -c "open -u username,password ftp.server.com/gets; get $downloads"

#derzeitiger Verzeichnisinhalt als old speichern
cp /var/log/gbcurrent /var/log/gbold

#alle dateien entpacken
unzip $downloads

Gruß,

JCB

 

[Wolfgang]

Hallo

#!/bin/sh
cd /pub/dnb/ftp/gb/fromgb

#liste holen und in lokaler Datei speichern
lftp -c "open -u username,password ftp.server.com/gets; ls C???M*.zip > /var/log/globalbatch"

#formatieren zu datei namens gbcurrent
cut -f 12 -d " " /var/log/globalbatch > /var/log/gbcurrent

#gbload, das was letztendlich runtergeladen werden soll als Liste erstellen
comm -3 /var/log/gbcurrent /var/log/gbold > /var/log/gbload

#gbload in variable einlesen

for i in `cat /var/log/gbload`; do

    downloads=$downloads$i;
done;


lftp -c "open -u username,password ftp.server.com/gets; get $downloads"

#derzeitiger Verzeichnisinhalt als old speichern
cp /var/log/gbcurrent /var/log/gbold

#alle dateien entpacken
unzip $downloads

Gruß,

JCB

Mein Vorschlag:
Ändere mal folgendes:

#gbload in variable einlesen

downloads="$(cat /var/log/gbload)"; #Alles in einem Array

for i in ${downloads[*]}; do

lftp -c "open -u username,password ftp.server.com/gets; get $i";
done
#derzeitiger Verzeichnisinhalt als old speichern
cp /var/log/gbcurrent /var/log/gbold

#alle dateien entpacken
for i in ${downloads[*]}; do
unzip "$i"
done

Nicht sehr elegant und ungeprüft, aber wenn du dir ein wenig man bash anschaust, wird es werden.
Ich würde das gesamte Problem etwas anders lösen.
Aber so verstehst du wenigstens um was es geht.
Falls du diese hässlichen Leerzeichen in Dateinamen hast, schau dir auch mal das Quoting der shell an.

HTH Gruß Wolfgang

 

[JCB]

Hallo Wolfgang,

vielen dank für deinen Input, jedoch ist dies nicht so elegant, da sich lftp dann x mal einloggen muss und das dauert bei der Ami-Kiste seeehr lange.

Das mit den Namen in den String wird nix?

Gruß,

Jens

 

[StyleWarZ]

kannst ja lftp durch wget ersetzen

 

[Xanti]

Hmm, wie sieht /var/log/gbload aus? Vielleicht tuts auch (ohne for-Schleife):

downloads=$(cat /var/log/gbload | tr "\n" " ")

Gruss, Xanti

 

[Wolfgang]

Hallo
Wenn du das gesamte Array übergeben willst, dann lass eben for i in weg und übergib das gesamte Array am Stück.

lftp -c "open -u username,password ftp.server.com/gets; get ${downloads[*]}";

Gruß Wolfgang

 

[JCB]

Hallo Xanti, Hallo Wolfgang,

die Vorschläge werde ich erstmal ausprobieren. Danke für Eure Hilfe.

Erfolgsmeldung am Dienstag

Gruß,

JCB

[edit]

So, es funktioniert wunderbar mit einer Mischung aus beiden Lösungen. Musste noch das Vergleichsscript anpassen, da auf dem Server Dateien automatisch wieder verschwinden und comm das auch als Unterschied interpretiert. Das ging aber gut wegzumachen.

Nochmals besten Dank an euch beide.

Gruß,

JCB