From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sun, 15 Jul 2007 22:18:24 +0000 (+0000)
Subject: Because plpgsql's scanner uses %option case-insensitive, flex's results could
X-Git-Url: http://waps.l3s.uni-hannover.de/gitweb/?a=commitdiff_plain;h=aa1ff6eba8671021f570ad0d1fcaa92558deeee8;p=users%2Fbernd%2Fpostgres.git

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.)
---

diff --git a/src/pl/plpgsql/src/Makefile b/src/pl/plpgsql/src/Makefile
index 5731048d76..9067d70969 100644
--- a/src/pl/plpgsql/src/Makefile
+++ b/src/pl/plpgsql/src/Makefile
@@ -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