AND instead of OR

Should be AND not OR, otherwise the backup for the DBs is done only when DBs are created and then never again.

requested review from @hahn

Hi Martin,

hey, es ist mein erster Tag :-)

if [ ! -z ${sSequenceLast} ] && [ "${sSequenceCurrent}" = "${sSequenceLast}" ]; then Wenn die erste Bedingung ungültig ist - also ${sSequenceLast} nicht existiert, dann kann man nicht mit einem AND dessen Wert mit dem Currentvalue vergleichen, ob diese gleich sind - das würde nie wahr.

Das Backup soll erfolgen, wenn

• noch nie kein Backup erfolgte --> sSequenceLast ist dann leer
• die letzte gesicherte SequenzID sSequenceLast ungleich der aktuell aus der DB geholten Sequenz-ID ist.

Die Bedingung für das Skip ist entsprechend umgekehrt. Es muss definitiv ein OR sein. Das erste ${sSequenceLast} müsste noch mit " gequotet sein - das könnte noch zu dem Syntax-Fehler führen, wenn sie leer ist. Ich tippe, das war eher das Problem.

Viele Grüsse, Axel

Hm, drum habe ich noch nichts gemacht...Fact ist, dass es mit dem || nicht läuft, respektive eben nur einmal, bis es eine SequenzID im Archiv gibt und dann nimmer. Ich habe dann (naiv wie ich bin) das mit dem && laufen lassen und siehe da, es wurden die DBs neu gesichert, weil bei 3/4 die Sequenz geändert hatte (Vergleich Atchiv/zu ist hat bei ist immer eine höhere Sequenz angezeigt. Dann war das mehr Zufall sagst du? Ich teste es gerne noch mit dem quote, also

if \[ ! -z "${sSequenceLast}" \] || \[ "${sSequenceCurrent}" = "${sSequenceLast}" \]; then

?

Wenn ich es nun nochmal so recht anschaue - reicht an sich die 2. Bedingung völlig aus, da die current SequenzID nie leer ist. Ich möchte wissen, wer das mal warum so programmiert hat.

Also so hier:

if [ "${sSequenceCurrent}" = "${sSequenceLast}" ]; then

added 1 commit

• fe730e69 - Update plugins/localdump/couchdb2.sh

Compare with previous version

merged

mentioned in commit ea080218