Re: [HACKERS] allowed user/db variables

Patch applied.  Thanks.

---------------------------------------------------------------------------


Joe Conway wrote:
> Bruce Momjian wrote:
> > Patch applied.  Thanks.
> >
>
> There was enough code drift since this patch, that a couple of bugs
> materialized.
>
> New items have been added to GucContext and GucSource enums, but of
> course they were not added to the corresponding GucContextName[] and
> GucSourceName[] arrays in the patch. Here's a new patch to fix the
> resulting bugs.
>
> Joe
>
>

> Index: src/backend/utils/misc/guc.c
> ===================================================================
> RCS file: /opt/src/cvs/pgsql-server/src/backend/utils/misc/guc.c,v
> retrieving revision 1.140
> diff -c -r1.140 guc.c
> *** src/backend/utils/misc/guc.c    27 Jul 2003 04:35:53 -0000    1.140
> --- src/backend/utils/misc/guc.c    27 Jul 2003 05:22:12 -0000
> ***************
> *** 156,162 ****
>
>
>   /*
> !  * Used for pg_settings. Keep in sync with config_type enum above
>    */
>   static char *config_type_name[] =
>   {
> --- 156,162 ----
>
>
>   /*
> !  * Used for pg_settings. Keep in sync with config_type enum in guc_tables.h
>    */
>   static char *config_type_name[] =
>   {
> ***************
> *** 176,181 ****
> --- 176,182 ----
>       "sighup",
>       "backend",
>       "super-user",
> +     "userlimit",
>       "user"
>   };
>
> ***************
> *** 188,193 ****
> --- 189,195 ----
>       "environment variable",
>       "configuration file",
>       "command line",
> +     "userstart",
>       "database",
>       "user",
>       "client",
> Index: src/include/utils/guc.h
> ===================================================================
> RCS file: /opt/src/cvs/pgsql-server/src/include/utils/guc.h,v
> retrieving revision 1.36
> diff -c -r1.36 guc.h
> *** src/include/utils/guc.h    27 Jul 2003 04:35:54 -0000    1.36
> --- src/include/utils/guc.h    27 Jul 2003 05:23:21 -0000
> ***************
> *** 52,57 ****
> --- 52,59 ----
>    * non-super users.
>    *
>    * USERSET options can be set by anyone any time.
> +  *
> +  * Keep in sync with GucContextName in guc.c
>    */
>   typedef enum
>   {
> Index: src/include/utils/guc_tables.h
> ===================================================================
> RCS file: /opt/src/cvs/pgsql-server/src/include/utils/guc_tables.h,v
> retrieving revision 1.2
> diff -c -r1.2 guc_tables.h
> *** src/include/utils/guc_tables.h    27 Jul 2003 04:35:54 -0000    1.2
> --- src/include/utils/guc_tables.h    27 Jul 2003 05:23:51 -0000
> ***************
> *** 58,63 ****
> --- 58,65 ----
>
>   /*
>    * GUC supports these types of variables:
> +  *
> +  * Keep in sync with config_type_name in guc.c
>    */
>   enum config_type
>   {

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073