Re: how to override config.site for lexer

[Stuart Henderson]

On 2012/04/03 19:05, Christopher Zimmermann wrote:
> On Tue, 3 Apr 2012 17:36:19 +0100
> Stuart Henderson  wrote:
> 
> > On 2012/04/03 17:31, Christopher Zimmermann wrote:
> > > CONFIGURE_ENV+= LEX=gflex
> > > 
> > > to the ports Makefile. This does not help because the configure
> > > script uses the "cached" value from the quick start macros in
> > > config.site. How can I convince configure to use gflex??
> > 
> > I forget the order of overrides, but try CONFIGURE_ARGS
> 
> I tried CONFIGURE_ARGS+="LEX=gflex", but this does no good:
> 
> checking for flex... (cached) flex

OK, in that case I'll read the configure script instead
and see what you need to do..

for ac_prog in flex lex
do
  # Extract the first word of "$ac_prog", so it can be a program name with args.
set dummy $ac_prog; ac_word=$2
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5
$as_echo_n "checking for $ac_word... " >&6; }
if ${ac_cv_prog_LEX+:} false; then :
  $as_echo_n "(cached) " >&6
else
  if test -n "$LEX"; then
  ac_cv_prog_LEX="$LEX" # Let the user override the test.
else
<...>

so, it looks like setting ac_cv_prog_LEX should do the trick.

> claws-mail has always been built with the flex in base. I just want to
> change it, because I noticed the check in autogen.sh and found the
> above entry in the changelog. Since I don't know what memory leak
> exactly they are talking about I find it difficult to check the base
> flex.
> 
> Christopher
> 

Re: how to override config.site for lexer

[Christopher Zimmermann]

On Tue, 3 Apr 2012 17:36:19 +0100
Stuart Henderson  wrote:

> On 2012/04/03 17:31, Christopher Zimmermann wrote:
> > CONFIGURE_ENV+= LEX=gflex
> > 
> > to the ports Makefile. This does not help because the configure
> > script uses the "cached" value from the quick start macros in
> > config.site. How can I convince configure to use gflex??
> 
> I forget the order of overrides, but try CONFIGURE_ARGS

I tried CONFIGURE_ARGS+="LEX=gflex", but this does no good:

checking for flex... (cached) flex

> > In case somebody wonders why ports gflex is required:
> > 
> > 2007-02-20 [colin]  2.7.2cvs56
> > 
> > * autogen.sh
> > * src/matcher_parser_parse.y
> > * src/matcher_parser_lex.l
> > Fix big memory leak in flex, and
> > require flex 2.5.31
> > 
> > 
> 
> Have you checked with flex in base anyway?
> Flex 2.5.30 had a major change to using m4 to generate the
> scanners which introduced a number of problems, and there's a
> chance that they require at least 2.5.31 to avoid one of those
> problems and perhaps 2.5.4 is actually OK. Or maybe not.  :)

claws-mail has always been built with the flex in base. I just want to
change it, because I noticed the check in autogen.sh and found the
above entry in the changelog. Since I don't know what memory leak
exactly they are talking about I find it difficult to check the base
flex.

Christopher