flex icon indicating copy to clipboard operation
flex copied to clipboard

remove hard-wired bison lookup, check for "GNU Bison" in $YACC's version output instead

Open GitMensch opened this issue 7 years ago • 3 comments

… fixes #386

additional:

  • don't check for GNU indent if indent-check already found no indent
  • use common wording to check "whether $BINARY is GNU stuff"

You may pass a full path and/or a different name for the Bison executable and/or additional options for the bison executable now.

Tested with Cygwin and Trisquel.

GitMensch avatar Sep 16 '18 18:09 GitMensch

On Monday, 17 September 2018, 1:04 am -0700, Simon Sobisch [email protected] wrote:

@westes your call: Do you want me to revert that part making configure.ac smaller (it doesn't count for the generated configure script which is much bigger) or stay with what the generated configure script normally does?

Please do so, yes.

westes avatar Sep 17 '18 10:09 westes

lines removed

GitMensch avatar Sep 17 '18 11:09 GitMensch

I know this is only 40 months old, but as we have a positive review by both @westes and @Explorer09 and a thorough discussion in the related issue - can you please merge this?

GitMensch avatar Jan 14 '22 07:01 GitMensch

Good that other issues were solved by merging PRs that had either a positive review or an actual request to be made (or both, like this one).... in the pure hope that this may be merged I'm sending out a ping @westes again.

GitMensch avatar Sep 26 '22 19:09 GitMensch

Friendly ping @westes as I'm cleaning up my branches...

GitMensch avatar Mar 01 '23 14:03 GitMensch