Thread: review of: collation for (expr)
Compiling on Ubuntu 10.04 LTS AMD64 on a GoGrid virtual machine from 2012-01-12 checkout.<br /><br />Bison upgraded to v2.5,and downgraded to v2.4.1<br /><br />Make process for both versions resulted in the following errors:<br /><br /> make[3]:Leaving directory `/usr/local/src/pgbuild/src/backend/catalog'<br />make -C parser gram.h<br />make[3]: Enteringdirectory `/usr/local/src/pgbuild/src/backend/parser'<br />/usr/local/bin/bison -d -o gram.c gram.y<br /> gram.y:conflicts: 370 reduce/reduce<br />gram.y: expected 0 reduce/reduce conflicts<br />gram.y:10482.27-10494.33: warning:rule useless in parser due to conflicts: func_expr: COLLATION FOR '(' a_expr ')'<br /> make[3]: *** [gram.c] Error1<br />make[3]: Leaving directory `/usr/local/src/pgbuild/src/backend/parser'<br />make[2]: *** [parser/gram.h] Error2<br />make[2]: Leaving directory `/usr/local/src/pgbuild/src/backend'<br /> make[1]: *** [all-backend-recurse] Error2<br />make[1]: Leaving directory `/usr/local/src/pgbuild/src'<br />make: *** [all-src-recurse] Error 2<br />
On tor, 2012-01-12 at 21:25 -0800, probabble wrote: > Compiling on Ubuntu 10.04 LTS AMD64 on a GoGrid virtual machine from > 2012-01-12 checkout. > > Bison upgraded to v2.5, and downgraded to v2.4.1 > > Make process for both versions resulted in the following errors: > > make[3]: Leaving directory `/usr/local/src/pgbuild/src/backend/catalog' > make -C parser gram.h > make[3]: Entering directory `/usr/local/src/pgbuild/src/backend/parser' > /usr/local/bin/bison -d -o gram.c gram.y > gram.y: conflicts: 370 reduce/reduce > gram.y: expected 0 reduce/reduce conflicts > gram.y:10482.27-10494.33: warning: rule useless in parser due to conflicts: > func_expr: COLLATION FOR '(' a_expr ')' > make[3]: *** [gram.c] Error 1 I can't reproduce that. In the meantime, attached is a re-merged version of the patch; the old version doesn't apply cleanly anymore.
Attachment
On Thu, Jan 19, 2012 at 1:50 PM, Peter Eisentraut <peter_e@gmx.net> wrote: > On tor, 2012-01-12 at 21:25 -0800, probabble wrote: >> Compiling on Ubuntu 10.04 LTS AMD64 on a GoGrid virtual machine from >> 2012-01-12 checkout. >> >> Bison upgraded to v2.5, and downgraded to v2.4.1 >> >> Make process for both versions resulted in the following errors: >> >> make[3]: Leaving directory `/usr/local/src/pgbuild/src/backend/catalog' >> make -C parser gram.h >> make[3]: Entering directory `/usr/local/src/pgbuild/src/backend/parser' >> /usr/local/bin/bison -d -o gram.c gram.y >> gram.y: conflicts: 370 reduce/reduce >> gram.y: expected 0 reduce/reduce conflicts >> gram.y:10482.27-10494.33: warning: rule useless in parser due to conflicts: >> func_expr: COLLATION FOR '(' a_expr ')' >> make[3]: *** [gram.c] Error 1 > > I can't reproduce that. > me neither > In the meantime, attached is a re-merged version of the patch; the old > version doesn't apply cleanly anymore. > besides a clash in the oid and the value of leakproof missing in the pg_proc entry, everything works fine. The only thing is that i don't see a reason for these includes in src/backend/utils/adt/misc.c: + #include "nodes/nodeFuncs.h" + #include "utils/lsyscache.h" -- Jaime Casanova www.2ndQuadrant.com Professional PostgreSQL: Soporte 24x7 y capacitación
On tor, 2012-03-01 at 20:30 -0500, Jaime Casanova wrote: > besides a clash in the oid and the value of leakproof missing in the > pg_proc entry, everything works fine. Fixed. > The only thing is that i don't see a reason for these includes in > src/backend/utils/adt/misc.c: > > + #include "nodes/nodeFuncs.h" > + #include "utils/lsyscache.h" lsyscache.h is necessary for type_is_collatable(), but the other was probably left over from a previous attempt. Committed with these changes.