[asterisk-bugs] [Asterisk 0014706]: [patch] A clean svn checkout needs bison to compile but configure does not check for it

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14706 
====================================================================== 
Reported By:                mvanbaak
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14706
Category:                   Core/BuildSystem
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for review
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-03-19 16:00 CDT
Last Modified:              2009-03-21 06:01 CDT
====================================================================== 
Summary:                    [patch] A clean svn checkout needs bison to compile
but configure does not check for it
Description: 
main/Makefile has a line to regenerate main/ast_expr2.c from the
main/ast_expr2.y
Normally this would only happen when the .c file is older then the .y
file.

While it might be so that both files have the same timestamp for the
Digium devs that do checkouts over LAN, this is not true for people with
normal DSL and/or Cable internet (and yes, there are even ppl with dialup
56kbps connections out there) and those checkouts will try to regenerate
the .c file.
We can fix this in three ways:
1) Add bison to the configure script
AC_PATH_PROG([BISON], [bison], [])
if test -z "$BISON"; then
 AC_MSG_ERROR([cannot find bison])
fi
2) rename the Makefile entry so it wont be run by default and people who
edited the .y file have a Makefiletarget to regen the .c file
3) Fix the core issue by renaming the .y file so it's checkedout before
the .c file.

Russell already said he did not want option 1 to go in because we 'do not
depend on bison'

====================================================================== 

---------------------------------------------------------------------- 
 (0102043) mvanbaak (manager) - 2009-03-21 06:01
 http://bugs.digium.com/view.php?id=14706#c102043 
---------------------------------------------------------------------- 
Updated patch that will actually bailout if bison is not installed. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-03-21 06:01 mvanbaak       Note Added: 0102043                          
======================================================================