Re: pkg/42509: lang/intercal wants to refer to yylineno which no longer exists

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR pkg/42509; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: gnats-bugs%NetBSD.org@localhost, pkg-manager%netbsd.org@localhost, 
gnats-admin%netbsd.org@localhost, 
        pkgsrc-bugs%netbsd.org@localhost, kre%munnari.OZ.AU@localhost
Cc: 
Subject: Re: pkg/42509: lang/intercal wants to refer to yylineno which no 
longer exists
Date: Wed, 23 Dec 2009 17:08:13 -0500

 On Dec 23,  8:35pm, kre%munnari.OZ.AU@localhost (Robert Elz) wrote:
 -- Subject: Re: pkg/42509: lang/intercal wants to refer to yylineno which no 
 
 |    | Add %option yylineno to the lex file?
 |  
 |  Thanks for the hint, that seemed simple enough (though I never found in
 |  the lex (flex) documentation any hint just where in the input sequence all
 |  the %option stuff it talks about should get put...) that I thought I should
 |  take a look at that and make a patch.
 |  
 |  But it turns out to be the exact opposite of what is wanted, intercal
 |  is (mostly) assuming it is using flex, which doesn't manage yylineno,
 |  and is handling it all for itself (I guess it uses that name for 
compatibility
 |  with other lex parsers, for the rest of the code).
 |  
 |  The problem turns out to be that patches/patch-ad "undoes" this
 |  assumption (essentially removes the declaration of yylineno and assumes
 |  that lex will provide it) - simply deleting patches/patch-ad fixed the
 |  problem for me.
 |  
 |  I assume though that someone needs that patch (which systems that pkgsrc
 |  runs on don't use a flex based lex?) - and that some more elegant solution
 |  than simply deleting the patch will be needed.
 
 Perhaps the portable solution is to rename the variable, since the program
 wants to manage it itself.
 
 christos