Peter Eisentraut <peter.eisentr...@2ndquadrant.com> writes: > On 8/19/16 1:04 AM, Masahiko Sawada wrote: >> I agree with adding this as an option and removing directory by default. >> And it looks good to me except for missing new line in usage output. >> >> printf(_(" -l, --label=LABEL set backup label\n")); >> + printf(_(" -n, --noclean do not clean up after errors")); >> printf(_(" -P, --progress show progress information\n"));
> Committed with that fix. Thanks. Hm, there was just a kerfuffle about spelling things like this "--no-clean" etc. I wasn't on board with removing existing spellings, but surely all newly added switches should use the "no-" spelling? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers