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 38092ee432f1602fd0070277c817d5559fb35e69..d6454a239b39e4ccedbc52995ff3f24d0ec82e0e 100644 (file)
--- 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