On Tuesday, March 11, 2025, vignesh C <vignes...@gmail.com> wrote: > > > Apart from the database, I believe the target file also needs to be > specified. Should we include this option also along with dbname: > + <group choice="plain"> > + <group choice="req"> > + <arg choice="plain"><option>-d</option></arg> > + <arg choice="plain"><option>--dbname</option></arg> > + </group> > + <replaceable>dbname</replaceable> > + </group> > > pg_recvlogical -U postgres --start -S test -d postgres > pg_recvlogical: error: no target file specified >
To reinforce my suggestion to “do nothing” here with the synopsis; your “show mandatory” policy would extend to “—slot” as well as communicating that ((create OR start) XOR drop) must also be specified. I am fine with this “show mandatory” policy in concept but doing it just here in lieu of an acceptable “no options” policy presently doesn’t make sense. There is debate, from me at least, whether such a command synopsis should include both short and long options in an alternate group or just short options, unless the option is a switch. You went with both here which I dislike. David J.
