Fix bogus %name-prefix option syntax in all our Bison files.

%name-prefix doesn't use an "=" sign according to the Bison docs, but it
silently accepted one anyway, until Bison 3.0.  This was originally a
typo of mine in commit 012abebab1bc72043f3f670bf32e91ae4ee04bd2, and we
seem to have slavishly copied the error into all the other grammar files.

Per report from Vik Fearing; analysis by Peter Eisentraut.

Back-patch to all active branches, since somebody might try to build
a back branch with up-to-date tools.

diff --git a/contrib/cube/cubeparse.y b/contrib/cube/cubeparse.y
index d7205b824cb5c0f21c913b5746552898d8590327..20aa81eadb66fdaeb94bd4349a426f71d922cc91 100644 (file)
--- a/contrib/cube/cubeparse.y
+++ b/contrib/cube/cubeparse.y
@@ -39,7 +39,7 @@ static NDBOX * write_point_as_box(char *s, int dim);
 /* BISON Declarations */
 %parse-param {NDBOX **result}
 %expect 0
-%name-prefix="cube_yy"
+%name-prefix "cube_yy"
 
 %token CUBEFLOAT O_PAREN C_PAREN O_BRACKET C_BRACKET COMMA
 %start box

diff --git a/contrib/seg/segparse.y b/contrib/seg/segparse.y
index 3fad9910bd54d382368a352d466ea2c203aa99c8..2311c1e63c0a428bd2655b9bdfedcff942ddb7d5 100644 (file)
--- a/contrib/seg/segparse.y
+++ b/contrib/seg/segparse.y
@@ -42,7 +42,7 @@ static char strbuf[25] = {
 /* BISON Declarations */
 %parse-param {SEG *result}
 %expect 0
-%name-prefix="seg_yy"
+%name-prefix "seg_yy"
 
 %union {
        struct BND {

diff --git a/src/backend/bootstrap/bootparse.y b/src/backend/bootstrap/bootparse.y
index eeffb0f990d7a47b306688fc48c7afdf2e7da712..0bd3aef7b85bbe7a586621aca606a2b3e29c05f8 100644 (file)
--- a/src/backend/bootstrap/bootparse.y
+++ b/src/backend/bootstrap/bootparse.y
@@ -93,7 +93,7 @@ static int num_columns_read = 0;
 %}
 
 %expect 0
-%name-prefix="boot_yy"
+%name-prefix "boot_yy"
 
 %union
 {

diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 8db1e2282064d5c357e6565d9241a07e6b95ba4c..0f9b72a3ff5b7d91a8ce2b3de289b2cad15a15a7 100644 (file)
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -148,7 +148,7 @@ static void processCASbits(int cas_bits, int location, const char *constrType,
 
 %pure-parser
 %expect 0
-%name-prefix="base_yy"
+%name-prefix "base_yy"
 %locations
 
 %parse-param {core_yyscan_t yyscanner}

diff --git a/src/backend/replication/repl_gram.y b/src/backend/replication/repl_gram.y
index b6cfdac1b66e09875b3368d1f2f9dec15bb3173e..72c984379c673a1c1e80315cdc17214c92b1f176 100644 (file)
--- a/src/backend/replication/repl_gram.y
+++ b/src/backend/replication/repl_gram.y
@@ -51,7 +51,7 @@ Node *replication_parse_result;
 %}
 
 %expect 0
-%name-prefix="replication_yy"
+%name-prefix "replication_yy"
 
 %union {
                char                                    *str;

diff --git a/src/interfaces/ecpg/preproc/ecpg.header b/src/interfaces/ecpg/preproc/ecpg.header
index 75538e3bd9601fdb28124001b925bb4cbef5b025..4db55eb8d7e24cf3fb8774fd375e283e83413427 100644 (file)
--- a/src/interfaces/ecpg/preproc/ecpg.header
+++ b/src/interfaces/ecpg/preproc/ecpg.header
@@ -557,7 +557,7 @@ add_typedef(char *name, char *dimension, char *length, enum ECPGttype type_enum,
 %}
 
 %expect 0
-%name-prefix="base_yy"
+%name-prefix "base_yy"
 %locations
 
 %union {

diff --git a/src/pl/plpgsql/src/gram.y b/src/pl/plpgsql/src/gram.y
index 4b2dce8cf0758ca05fc59c7a345c3d1a9c031085..52cbfa60af908fee637706e3835ff7b2400394ee 100644 (file)
--- a/src/pl/plpgsql/src/gram.y
+++ b/src/pl/plpgsql/src/gram.y
@@ -110,7 +110,7 @@ static      List                    *read_raise_options(void);
 %}
 
 %expect 0
-%name-prefix="plpgsql_yy"
+%name-prefix "plpgsql_yy"
 %locations
 
 %union {

diff --git a/src/test/isolation/specparse.y b/src/test/isolation/specparse.y
index bf3a9f3b5058c4c08b06870b39e4bc9fa458b205..c10ec039724c18016c221597ee060b59a52992ae 100644 (file)
--- a/src/test/isolation/specparse.y
+++ b/src/test/isolation/specparse.y
@@ -20,7 +20,7 @@ TestSpec              parseresult;                    /* result of parsing is left here */
 %}
 
 %expect 0
-%name-prefix="spec_yy"
+%name-prefix "spec_yy"
 
 %union
 {