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(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073