Cronjob? Ser ni något fel?

#!/bin/sh
BACKUP_PATH="/home/mc/backups"
MC_PATH="/home/mc/mc/Minecraft"
TGZ_FILE="backup_$(date +%Y%m%d_%H%M%S).tar.gz"
SCREEN_NAME="MC"

# När man börjar upprepa sig så är det ofta bra att skapa en funktion så att
# man undviker skrivfel och samtidigt håller skriptet mer lättläst.
mc_cmd() {
	# Man får hålla tungan rätt i mun för att hänga med i hur
	# citationstecknen fungerar här, men det ska vara all right för
	# närvarande. $1 är helt enkelt det första argumentet när funktionen
	# kallas.
	screen -S "$SCREEN_NAME" -X eval 'stuff "/'"$1"'\015"'
}

mc_cmd save-off
mc_cmd save-all
tar cpzf "$BACKUP_PATH/$TGZ_FILE" "$MC_PATH"
mc_cmd save-on

# Denna bit ser inte speciellt robust ut i mina ögon. Man ska alltid vara
# defensiv när det gäller att skicka filnamn genom pipes, då filnamn _kan_
# innehålla saker som kan överraska (exempelvis radbrytningar och annat). Även
# om man "vet" att de inte gör det så är det inte bra praxis att lära sig
# olater, när det finns mer robusta sätt tillgängliga. Ersätt hellre med
# exempelvis `find` som rensar allt äldre än 24 dagar (här baserat på dess
# datumstämpel snarare än dess filnamn), om det är det du vill.
#cd /home/mc/backups
#ls -t | sed -e '1,24d' | xargs -d '\n' rm
find "$BACKUP_PATH" -mindepth 1 -mtime +24 -delete
# Läs `man find` för att se vad ovanstående kommando gör!

# Filuppladdning löses troligen enklare om du har ett shellkonto någonstans och
# kan köra `rsync` som nämndes tidigare i tråden, men det är också lösbart
# genom FTP.
#
# Så som du har skrivit nu är inte vettigt dock. Varje rad i ett shellskript
# exekveras av tolken, så det som du tror hanteras av FTP-programmet
# försöker skalet i stället köra som kommandon, vilket lär ge en del
# felmeddelanden i bästa fall, och göra något oväntat i sämsta fall.
#
# Det är dock nästan möjligt att göra som du skriver, men kommandona behöver
# skickas in som en sträng. Vi kan göra det med "heredoc"-syntax:
#ftp <<EOF
#open minip
#user mittanv mittlösenord
#put "$BACKUP_PATH/$TGZ_FILE"
#EOF
# Alternativt kan man här använda Curl:
curl --upload-file "$BACKUP_PATH/$TGZ_FILE" ftp://minip --user mittanv:mittlösenord
# Om detta körs på en "delad dator" så finns det en säkerhetsbrist i att ange
# inloggningsuppgifter direkt på kommandoraden. Bättre är att använda en fil
# med dessa uppgifter lagrade; se växeln `-n` till Curl för att använda
# `~/.netrc` till detta (denna fil går även att använda för `ftp`).
#
# Man kan även titta på `lftp` och `ncftp` om man vill göra mer avancerade
# saker över FTP.
#
# Notera att det allmänt är lite vanskligt att behöva ange lösenord i klartext,
# och FTP är på egen hand väldigt osäkert. Vanligare är att använda
# SSH-inloggning med nycklar, om nu fjärrservern man vill köra backup till är
# en "vettig dator" som förstår SSH :-) .

if ! [ -d "$BACKUP_PATH" ]; then
	printf '%s: fel: "%s" är inte en giltig katalog! Avbryter.\n' "${0##*/}" "$BACKUP_PATH" 1>&2
	exit 2
fi
find "$BACKUP_PATH" -mindepth 1 -mtime +24 -delete

find "$BACKUP_PATH" -mindepth 1 -maxdepth 1 -type f -mtime +24 -delete