[meteo-vm2-to-bufr] Exit status != 0 on encode/decode errors

[mnuccioarpae]

@edigiacomo nulla in contrario per aggiornare aff9b6e e farne il merge?

[edigiacomo]

Per me va benissimo! L'unica cosa su cui potremmo discutere è se tenere come comportamento di default quello attuale o meno.

[mnuccioarpae]

A me sembra più intuitivo che l'exit status non sia 0 quando ci sono errori. Non a caso, mi sono accorto di questo problema perché il comando non falliva, quindi log_on_error non mostrava gli errori di decodifica.

Il problema è che si tratta di una breaking change, quindi qualche procedura potrebbe generare errori mai visti prima. L'unico che mi viene in mente è se l'output di meteo-vm2-to-bufr è passato in pipe | ad un altro comando e si è fatto set -o pipefail o si controlla PIPESTATUS.

Non mi sembra un grosso problema perché dovrebbero essere errori facilmente risolvibili, per esempio filtrando le variabili con problemi (non mi piace, ma l'effetto finale credo che sia lo stesso, visto che le variabili non decodificabili non compaiono comunque nell'output).

[edigiacomo]

Andata, procedi pure, grazie!

[mnuccioarpae]

Andata, procedi pure, grazie!

quale delle due?

1. uso di opzioni proposto in aff9b6e
2. exit status != 0 se ci sono errori di conversione (senza aggiungere una opzione)

[edigiacomo]

Scusami, il mio "andata" era riferito alla tua proposta di usare exit status != 0 quando ci sono errori.