Greetings,

12-08-2004 Perşembe günü saat 18:32 sularında, Tom Lane dedi ki:
ntufar <[EMAIL PROTECTED]> writes:
> > flex (flex version 2.5.4) incorporates case-insensitivity in it's
> > state tables because if I run flex stage with LANG=C everything
> > works fine.
>
> Ick.  That is of course why it worked for me when I tested it :-(
>
> > A quick and dirty fix could be implemented by placing
> >      LANG=C
> >      export LANG
> > in file src/pl/plpgsql/src/Makefile before calling flex.
>
> This is probably what we'd better do.  Otherwise we have
> build-context-dependency in the system's behavior, which is bad.
>
I attached a diff of fix that adds LANG=C; before call to $(FLEX).
Fixes the problem here but I don't know if adding environment variable
assignment like this is appropriate. I am not too fluent in PostgreSQL
build environment and do not know where one can put a global deffinition
you are talking below.

Peter, any thoughts on this one way or the other?  At the moment
> plpgsql's scan.l seems to be the only use of '%option
case-insensitive'
> but we have enough flex lexers laying about that I wouldn't be
surprised
> to have this same risk elsewhere.  Is it reasonable to try to force
> LANG=C in some global fashion during the build?
>
>                       regards, tom lane
>
Best regards,
Nicolai Tufar

"w^ÆÊÜş?¦Z`²©²·?1©~)^	'â?ïé®?ŞrÛ?rû+¢?¦*?Ç«½êÿ²·?¦_é??,ª_ì­ÏÌjG??W¯­ëk?ëâ?
޾+"¢}véØ?}ÊõÛ£?çâ?ë+súeş?i?Ê¥şÊÜüƤyø¥{rn?m4ã_v×}tÓM5Û«+súeş?i?Ê¥şÊÜüƤyø¥{]?º
´Ó?|ã?7ÓM4ï¿7²·?¿é?Ç?Ëjyb}×?±±±°Húe¦*?¶¬{,¨z[?+,?x?ì^?Ø?ï¿7²·?¿é?Ç?Ëjyb}×?±,F!KqKqKd?¦Z`²©[jDzÊ?¥±é¢²È§?ù^Åéİ
---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faqs/FAQ.html

Reply via email to