At Wed, 14 Jun 2023 00:49:39 +0900, torikoshia <torikos...@oss.nttdata.com>
wrote in
> On 2023-06-12 16:33, Michael Paquier wrote:
> > On Fri, Jun 09, 2023 at 12:32:15AM +0900, Fujii Masao wrote:
> Thanks for reviewing!
>
> >> printf(_(" -n, --dry-run dry run, show the names of the files that
> >> would be removed\n"));
> >> + printf(_(" -b, --clean-backup-history clean up files including
> >> backup history files\n"));
> >> Shouldn't -b option be placed in alphabetical order?
> > +1.
>
> Modified the place.
- printf(_(" -d generate debug output (verbose mode)\n"));
- printf(_(" -n dry run, show the names of the files that
would be removed\n"));
- printf(_(" -V, --version output version information, then exit\n"));
- printf(_(" -x EXT clean up files if they have this
extension\n"));
- printf(_(" -?, --help show this help, then exit\n"));
+ printf(_(" -d, --debug generate debug output (verbose
mode)\n"));
+ printf(_(" -n, --dry-run dry run, show the names of the
files that would be removed\n"));
+ printf(_(" -V, --version output version information,
then exit\n"));
+ printf(_(" -x, --strip-extension=EXT strip this extention before
identifying files fo clean up\n"));
+ printf(_(" -?, --help show this help, then exit\n"));
After this change, some of these lines corss the boundary of the 80
columns width. (is that policy viable noadays? I am usually working
using terminal windows with such a width..) It's somewhat unrelated to
this patch, but a help line a few lines further down also exceeds the
width. We could shorten it by removing the "/mnt/server" portion, but
I'm not sure if it's worth doing.
> Or for use as a standalone archive cleaner:
> e.g.
> pg_archivecleanup /mnt/server/archiverdir
> 000000010000000000000010.00000020.backup
+ printf(_(" -x, --strip-extension=EXT strip this extention before
identifying files fo clean up\n"));
s/fo/for/ ?
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
