From 78c84ad49eeb12c67797e5acb3cf637a59601ce2 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 15 Jul 2007 22:18:24 +0000 Subject: [PATCH] Because plpgsql's scanner uses %option case-insensitive, flex's results could theoretically vary depending on what the compile-time locale setting is. Hence, force it to see LC_CTYPE=C to ensure consistent build results. (It's likely that this makes no difference in practice, since our specification for "identifier" surely includes both ends of any possible uppercase/lowercase pair anyway. But it should silence warnings about ambiguous character classes that are reported by some buildfarm members.) --- src/pl/plpgsql/src/Makefile | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/pl/plpgsql/src/Makefile b/src/pl/plpgsql/src/Makefile index 38092ee432f..d6454a239b3 100644 --- a/src/pl/plpgsql/src/Makefile +++ b/src/pl/plpgsql/src/Makefile @@ -2,7 +2,7 @@ # # Makefile for the plpgsql shared object # -# $PostgreSQL: pgsql/src/pl/plpgsql/src/Makefile,v 1.30 2007/02/09 15:56:00 petere Exp $ +# $PostgreSQL: pgsql/src/pl/plpgsql/src/Makefile,v 1.31 2007/07/15 22:18:24 tgl Exp $ # #------------------------------------------------------------------------- @@ -67,9 +67,13 @@ else @$(missing) bison $< $@ endif +# Because we use %option case-insensitive, flex's results could vary +# depending on what the compile-time locale setting is. Hence, force +# it to see LC_CTYPE=C to ensure consistent build results. + $(srcdir)/pl_scan.c: scan.l ifdef FLEX - $(FLEX) $(FLEXFLAGS) -o'$@' $< + LC_CTYPE=C $(FLEX) $(FLEXFLAGS) -o'$@' $< else @$(missing) flex $< $@ endif -- 2.39.5