Re: [PATCHES] Patch for listing runtime option details through server

2003-07-02 Thread Bruce Momjian
Tom has applied your patch, with edits. Thanks. --- Aizaz Ahmed wrote: > > The attached patch adds the --long-help option to the server executable. > This option displays all the available runtime options for that > partic

Re: [PATCHES] Patch for listing runtime option details through server executable (pg_guc)

2003-07-02 Thread Tom Lane
I've committed Aizaz's patch, with some editorial revisions. Peter Eisentraut <[EMAIL PROTECTED]> writes: > If the option is named --long-help, I'd expect a longer version of > --help, which this is not. The name should probably involve "help" > and "config" to make it clearer what you get. I us

Re: [PATCHES] Patch for listing runtime option details through server executable (pg_guc)

2003-06-30 Thread aahmed
Quoting Tom Lane <[EMAIL PROTECTED]>: > > Should options not for general use (e.g., session_auth_is_superuser) > > be hidden from this tool? Are they? What other provisions of this > > kind does this tool make? > > The original intent of the standalone tool was to display stuff that > could use

Re: [PATCHES] Patch for listing runtime option details through server executable (pg_guc)

2003-06-30 Thread Tom Lane
Peter Eisentraut <[EMAIL PROTECTED]> writes: > If the option is named --long-help, I'd expect a longer version of > --help, which this is not. The name should probably involve "help" > and "config" to make it clearer what you get. (Personally, I think > "help" should go before the qualifying word

Re: [PATCHES] Patch for listing runtime option details through server

2003-06-30 Thread Peter Eisentraut
Aizaz Ahmed writes: > The attached patch adds the --long-help option to the server executable. Conceptual comments: If the option is named --long-help, I'd expect a longer version of --help, which this is not. The name should probably involve "help" and "config" to make it clearer what you get.