Re: Fix help option of contrib/oid2name

On 2018/08/21 12:57, Tatsuro Yamada wrote:
> On 2018/08/21 12:40, Michael Paquier wrote:
>> On Tue, Aug 21, 2018 at 12:26:15PM +0900, Tatsuro Yamada wrote:
>>> BTW, can I add the patch to the Commitfest September?
>>
>> You should.
> 
> Thanks, I'll do that.
> I'll send 2 patches in this week, probably.
I revised the patch and created new tap tests.

fix_oid2name_wip3.patch:
   bug fix
     - Remove "-P password" option from the document
   improvements
     - Divide "Options section" into "Options" and "Connection Options" sections in
       the help message (--help)
     - Add long options
     - Add "-H host" and "-h host" options to the help message and the document
     - Add environment variable (PGHOST, PGPORT and PGUSER) to the document

fix_vacuumlo_wip3.patch:
   improvements
     - Add long options
     - Add environment variable (PGHOST, PGPORT and PGUSER) to the document

Regarding to Long options,
I read the following both codes and I understand meanings of short options, so I
leave all long options in patches not only for connection options.

oid2name.c
	/* these are the opts structures for command line params */
	struct options
	{
	    eary       *tables;
	    eary       *oids;
	    eary       *filenodes;

	    bool        quiet;
	    bool        systables;
	    bool        indexes;
	    bool        nodb;
	    bool        extended;
	    bool        tablespaces;

	    char       *dbname;
	    char       *hostname;
	    char       *port;
	    char       *username;
	    const char *progname;
	};

vacuumlo.c
	struct _param
	{
	    char       *pg_user;
	    enum trivalue pg_prompt;
	    char       *pg_port;
	    char       *pg_host;
	    const char *progname;
	    int         verbose;
	    int         dry_run;
	    long        transaction_limit;
	};


Following TAP tests is for checking command-line options but I coudn't add some
test cases such as it is required argument and mixed varaiety of options.
I'm not sure about naming rule of tap test, so I added 300 and 301 to names as
temporarily. Please rename these files to more suitable names.

   300_oid2name.pl
   301_vacuumlo.pl

Please find attached files.

Thanks,
Tatsuro Yamada