[BAT]Script zit vast in een loop?

[ThinkPad]

Dit: http://hazymat.co.uk/2011/08/amazing-robocopy/ script leek mij wel handig om hier thuis te gebruiken voor m'n backups.

Maar op de een of andere manier zit hij vast in een loop? Wat ik te zien krijg zodra ik hem run:

IP Replied! Checking connection stability...1 of 6
IP Replied! Checking connection stability...2 of 6
IP Replied! Checking connection stability...3 of 6
IP Replied! Checking connection stability...4 of 6
IP Replied! Checking connection stability...5 of 6
Connection appears stable!
Now checking fileshare
Fileshare is visible. Good to go. Starting copying.
robocopy "D:\Mijn afbeeldingen" "\\192.168.1.137 \BACKUP\PCFRANK\Mijn afbeeldingen" /MIR /ETA /XJ /R:10 /W:10
IP Replied! Checking connection stability...1 of 6
IP Replied! Checking connection stability...2 of 6

De laatste 2 regels gaat het dus fout, hij begint weer opnieuw dus. En dit gaat eindeloos zo door. Iemand die ergens de fout ziet? Ik zie hem namelijk niet.

Dit is het script zoals ik het nu heb:

Visual Basic:

: This script does a basic robocopy, but also it does the following: 
: - test connectivity to the machine (ping). Send WOL Magic Packet if it doesn't respond. 
: - after WOL, wait until the machine appears on the network 
: - regardless of whether or not we got the machine up using WOL, we still verify a level of ping response consistency before continuing 
: - verify the network path is visible before continuing 
: - prompt at the start whether you want to shutdown the machine when finished 
: - no further prompts during the process 

@ECHO OFF 

: SET THE FOLLOWING VARIABLES 

: Set robocopy destination into two variables. They are used individually to test CIFS and PING connectivity then combined to insert into robocopy command 
: We'll strip quotes from the outsides of these, so feel free to use quotes around each varilable - or not. 
Set remotemachine=192.168.1.137 
Set copysource=D:\Mijn afbeeldingen\ 
Set copytoshare=BACKUP\%COMPUTERNAME%\Mijn afbeeldingen

: Time to wait after sending wol packet, before bothering to try to do anything else (approx startup time of remote machine) 
Set timetowol=30 

: If, after sending wol and waiting, there's still no response, we'll wait 1 second and try again. 
: This is the total number of tries. TBH, may as well set this really high and Ctrl-C if you get bored. 
Set pingfaillimit=25 

: What do you consider is a good number of ping receipts to get back before deeming your connection to remote machine is stable? 1 = impatient. 10000 = paranoid. 10 = normal. 
Set stabilitysatisfaction=6 

: Once stability, by the definition of how many pings specified, is attained, we check the copy-to network path is available 
: Frankly, if it isn't, it probably won't become available. And you'll have to figure out the problem separately. 
: But this gives us the option to keep trying x number of times before continuing. 
: Note: this number doesn't correspond to an amount of time. Windows is unpredictable when trying to check fileshares. 
Set filesharefaillimit=15 

: NO MORE VARIABLES TO SET NOW 

Set consecutivepingcheckcount=1 
Set consecutivepingfailcount=1 
Set filesharetestcount=1 

:pingandcheck 

ping /n 2 %remotemachine% | find "TTL=" >nul 
if %errorlevel% == 0 goto reply 

@echo No Reply on that IP! Tried %consecutivepingfailcount% of %pingfaillimit% times 

Set consecutivepingcheckcount=1 
Set /A consecutivepingfailcount+=1 

IF %consecutivepingfailcount% == %pingfaillimit% ( 
@echo We didn't get very far did we? 
@echo I waited, but nothing! 
@echo Increase the pingfaillimit variable? 
GOTO fin 
) 
goto pingandcheck 

:reply 
@echo IP Replied! Checking connection stability... %consecutivepingcheckcount% of %stabilitysatisfaction% 
Set /A consecutivepingcheckcount+=1 
IF %consecutivepingcheckcount% == %stabilitysatisfaction% ( 
@echo Connection appears stable! 
GOTO checkfileshare 
) 
GOTO pingandcheck 

:checkfileshare 
@echo Now checking fileshare 
IF EXIST \\%remotemachine%\%copytoshare% ( 
@echo Fileshare is visible. Good to go. Starting copying. 
GOTO docopy 
) 
@echo Couldn't find fileshare - tried %filesharetestcount% of %filesharefaillimit% times. 
Set /A filesharetestcount+=1 
ping 127.0.0.1 -n 2 >null 
IF %filesharetestcount% == %filesharefaillimit% ( 
@echo Failed to find the fileshare. Oh no! 
@echo Maybe verify the fileshare is accessible yourself? 
GOTO fin 
) 
goto checkfileshare 

:docopy 
:first three lines strip quotes if found then combine the machine name and share to give path 
for /f "useback tokens=*" %%a in ('%remotemachine%') do set remotemachine=%%~a 
for /f "useback tokens=*" %%a in ('%copytoshare%') do set copytoshare=%%~a 
Set destination=\\%remotemachine%\%copytoshare% 
ECHO on 
robocopy "%copysource%" "%destination%" /MIR /ETA /XJ /R:10 /W:10 

:fin

[DeTeraarist]

je script heet robocopy.bat?

[ThinkPad]

DeTeraarist schreef op dinsdag 18 september 2012 @ 21:17:
je script heet robocopy.bat?

Ja...."robocopy.bat". Renamen naar iets anders did the trick. Ik voel mij heeeeeel dom nu Logisch dat hij dan het script weer aanroept idd

[Hero of Time]

Op Linux geen last van, daar kan je robocopy en robocopy.sh hebben, en robocopy.ksh en robocopy.zsh en, en, tja, noem maar op. Desnoods noem je 't ook robocopy, als 't maar niet in je $PATH staat waar de werkelijke executable is, is er niets aan de hand. En anders doet 't niets . Nouja, niets, forkbomb is redelijk aanwezig als je niet oplet

* Hero of Time vind 't leuk op Linux scripten, geeft veel leukere effecten om een script op 1 distro te maken en uit te voeren op een ander

[ Voor 6% gewijzigd door Hero of Time op 18-09-2012 22:06 ]

[RaZ]

In Windows kan het ook gewoon. Er wordt gewoon geen extentie meegegeven, dat aanpassen naar robocopy.exe (van uitgaand dat het een .exe is), dan zal die nooit de .bat starten.

[ThinkPad]

Ik zit nu alleen nog vast wanneer ik mappen met een spatie in de naam (Mijn documenten) wil kopieren. Robocopy gaat dan over de zeik....

VBScript:

'Eerste stuk van het script
Set remotemachine=192.168.1.137
Set copyto="BACKUP\%COMPUTERNAME%\Mijn documenten\"
Set copyfrom="D:\Mijn documenten\"

'Onderste deel van het script
:docopy
:first three lines strip quotes if found then combine the machine name and share to give path
for /f "useback tokens=*" %%a in ('%remotemachine%') do set remotemachine=%%~a
for /f "useback tokens=*" %%a in ('%copyto%') do set copyto=%%~a
Set destination=\\%remotemachine%\%copyto%
ECHO on
robocopy %copyfrom% %destination% /MIR /ETA /XJ /R:10 /W:10 /LOG+:backup_logfile.log

Het heeft iets met de quotes te maken, maar ik snap niet precies wat. Heb al geprobeerd om bij de SET de quotes weg te halen of juist te plaatsen, of de "for /f" regels uit te commentaren, maar helpt niet. Robocopy geeft dit als output:

Bron : D:\Mijn documenten" \192.168.1.137\BACKUP\FRANKPC\Mijn documenten \MIR \ETA \XJ \R:10 \W:10 \LOG:backup_logfile.log\
Doel -

De regel "Bron" gaat dus helemaal over de zeik

[DukeBox]

geen quotes in je set maar in je command gebruiken.
Dus set copyto=backup\%computername%\My Documents\
en
robocopy "%copyfrom%" ...

[ Voor 44% gewijzigd door DukeBox op 18-09-2012 22:48 ]

[Hero of Time]

RaZ schreef op dinsdag 18 september 2012 @ 22:10:
In Windows kan het ook gewoon. Er wordt gewoon geen extentie meegegeven, dat aanpassen naar robocopy.exe (van uitgaand dat het een .exe is), dan zal die nooit de .bat starten.

De grap met Linux is juist dat als een bestand een extensie heeft, je die moet opgeven. Als je je vasthoud aan een principe dat je scripts altijd een extensie geeft, zal je dit probleem dus nooit krijgen . En anders kan je net als bij Windows het volledige pad naar de executable geven.


On-topic:
Quotes zijn echt een ramp soms in scripts. De ene keer moet 't wel, de andere keer niet, dan gaat het goed, dan weer niet. Het is daarom handig om je variabelen eerst te controleren via een echo en als de output je niet aan staat, bijvoorbeeld er zijn geen quotes, dan zet je quotes om je variabele om 't zo wel te krijgen. Trial en error. Je zou dan iets als dit krijgen:

code:

echo executing command %var1% %var2% etc.
: of, als 't niet goed gaat
echo executing command "%var1%" "%var2" etc.

De boel heb je al in quotes staan in je voorbeeld. Waarom het dan niet werkt, is mij een raadsel, maar nou heb ik ook geen ervaring met batch scripting. Geef mij maar bash .

Btw, waarom zet je eerst variabele %copytoshare%, gebruik je 'm een paar keer, om vervolgens aan het eind in je commando 'm niet te gebruiken. Waar komt %destination% opeens vandaan?

[ThinkPad]

DukeBox schreef op dinsdag 18 september 2012 @ 22:47:
geen quotes in je set maar in je command gebruiken.
Dus set copyto=backup\%computername%\My Documents\
en
robocopy "%copyfrom%" ...

Zal het thuis proberen (ben nu aan het werk).

Hero of Time schreef op dinsdag 18 september 2012 @ 23:14:
[...]
[..]
Btw, waarom zet je eerst variabele %copytoshare%, gebruik je 'm een paar keer, om vervolgens aan het eind in je commando 'm niet te gebruiken. Waar komt %destination% opeens vandaan?

Thanks %destination%:
Set destination=\\%remotemachine%\%copytoshare%