[ragel-users] [PATCH 3/6] Make it possible to enable/disable parsers generation at ./configure

Adrian Thurston thurs... at cs.queensu.ca
Mon Oct 6 17:24:05 UTC 2008


I don't see why anyone would need to use this. Either you're doing 
development on ragel and you're working out of SVN or you're grabbing a 
release for building and installing. Maybe someone might want to do 
development on an older version but I think that would be pretty rare.

Adrian

Diego 'Flameeyes' Pettenò wrote:
> Instead of having to edit the configure.in file to disable parsers,
> make it possible to disable it at ./configure stage, like any good
> autoconf project should do.
> ---
> 
>  configure.in       |   12 +++++++-----
>  ragel/Makefile.in  |    6 +++---
>  redfsm/Makefile.in |    2 +-
>  3 files changed, 11 insertions(+), 9 deletions(-)
> 
> diff --git a/configure.in b/configure.in
> index cc25f2e..3995551 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -21,10 +21,12 @@ dnl   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
>  AC_INIT(ragel/main.cpp)
>  AC_CONFIG_HEADER(common/config.h)
>  
> -dnl Set to true if build system should generate parsers from ragel, kelbt, and
> -dnl gperf sources. Set to false if generated files are included and not to be
> -dnl built (production).
> -AC_SUBST(BUILD_PARSERS,true)
> +dnl You need to enable this unless the generated files are included
> +dnl and don't need to be built.
> +AC_ARG_ENABLE([parsers],
> +  AS_HELP_STRING([--disable-parsers], [Disable building of parsers with ragel, kelbt and gperf]),
> +  [], [enable_parsers=yes])
> +AC_SUBST([enable_parsers])
>  
>  dnl Checks for programs.
>  AC_PROG_CC
> @@ -38,7 +40,7 @@ AC_LANG_CPLUSPLUS
>  dnl Check for definition of MAKE.
>  AC_PROG_MAKE_SET
>  
> -if test $BUILD_PARSERS = true; then
> +if test $enable_parsers = yes; then
>  
>  dnl Check for Ragel
>  AC_CHECK_PROG(RAGEL, ragel, ragel)
> diff --git a/ragel/Makefile.in b/ragel/Makefile.in
> index b6397dc..2a6b5da 100644
> --- a/ragel/Makefile.in
> +++ b/ragel/Makefile.in
> @@ -45,7 +45,7 @@ MINGW_LIBS = -lpsapi
>  #*************************************
>  
>  prefix = @prefix@
> -BUILD_PARSERS = @BUILD_PARSERS@
> +BUILD_PARSERS = @enable_parsers@
>  EXEEXT = @EXEEXT@
>  CXX = @CXX@
>  
> @@ -63,7 +63,7 @@ all: ragel$(EXEEXT)
>  ragel$(EXEEXT): $(GEN_SRC) $(OBJS) $(RAGEL_LIBS)
>  	$(CXX) $(LDFLAGS) -o $@ $(OBJS) $(LIBS)
>  
> -ifeq ($(BUILD_PARSERS),true)
> +ifeq ($(BUILD_PARSERS),yes)
>  
>  rlparse.h: rlparse.kh
>  	kelbt -o $@ $<
> @@ -87,7 +87,7 @@ endif
>  distclean: clean
>  	rm -f Makefile 
>  
> -ifeq ($(BUILD_PARSERS),true)
> +ifeq ($(BUILD_PARSERS),yes)
>  EXTRA_CLEAN = $(GEN_SRC)
>  endif
>  
> diff --git a/redfsm/Makefile.in b/redfsm/Makefile.in
> index a4c6737..951de79 100644
> --- a/redfsm/Makefile.in
> +++ b/redfsm/Makefile.in
> @@ -28,7 +28,7 @@ CC_SRCS = redfsm.cpp gendata.cpp xmltags.cpp xmlscan.cpp xmlparse.cpp
>  
>  GEN_SRC = xmltags.cpp xmlscan.cpp xmlparse.cpp xmlparse.h
>  
> -BUILD_PARSERS = @BUILD_PARSERS@
> +BUILD_PARSERS = @enable_parsers@
>  
>  #*************************************
>  
> 
> 
> 



More information about the ragel-users mailing list