[rzezimieszek_]

Napisałem taki skrypcik do backupów, chciałem zapytać co sądzicie o liniach 33 i 46, czy jest to poprawna forma? Poniżej ogólna, skrócona wersja, dodatkowo są opcje miesięczny i roczny analogiczne do zamieszczonego tygodniowego backupu. Debian 7

https://pastebin.com/9t9907qt

#shell #bash #linux

[maniac777]

@rzezimieszek_:
33. Zamiast -exec uzylbym -delete
46. Prawdopodobnie bledne i chyba nie zadziala tylko zostanie zinterpretowane jako tekst.

[adrpan]

@rzezimieszek_: a masz mocny procek na serwerze? jak tak to proponuje kompresować przez 7zip, bazę 1GB do 90MB, można wtedy nawet się pokusić o upload w jakąś chmurę, czy innego ftp'ka

[Kryspin013]

@rzezimieszek_: warunek w 46 lini albo się wysypie albo zawsze zwróci true.

Piszę z głowy więc mogę się mylić ale najlepiej jakbyś zapisał

if [ "$(find $mainweeklydest -mtime +7 | wc -l)" -gt 1 ] ; then

[rzezimieszek_]

@adrpan niestety na razie nie, myślałem kiedyś o NAS lokalnie na te backupy, ale za dużo przeszkód.

@Kryspin013 dzięki bardzo, będę kombinował i dam znać

[Kryspin013]

@rzezimieszek_: W zasadzie to to twoje też powinno zadziałać ale niestety warunek masz zawsze spełniony bo porównujesz łańcuchy a nie wartości, np tutaj warunek jest zawsze spełniony:

if echo 1 | cat | wc -l > 2 ; then echo tak ; fi

[rzezimieszek_]

@Kryspin013 na pewno do tego ifa muszę jeszcze dorzucić opcję gdy nie ma żadnego backupu i to po prostu robię w ten sposób
if [... ] || [... ];

[Kryspin013]

@rzezimieszek_: oho, z tymi warunkami dla ifa to też potestuj czy działa wcześniej bo te wszystkie andy i ory lubią losowo działać w zależności od składni. Pamiętam że kiedyś coś co było jako || miało zupełnie inne działanie nić or zaposany jako flaga.