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.)
This commit is contained in:
Tom Lane 2007-07-15 22:18:24 +00:00
parent 816ff27f60
commit 78c84ad49e
1 changed files with 6 additions and 2 deletions

View File

@ -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