Ho, On Monday 10 March 2014 17:05, Tom Hacohen wrote : > Hey, > > The reason why I haven't added it until now, is that we do > auto-detection any more (i.e you have to enable/disable), and I didn't > want to pollute the configure script with this option. Not sure which is > worse. I know about the no more auto-detect rule, but thought it is feature a centric rule, that for me is an optional dev tool (luckilly we don't need to (en|dis)able ar disable gcc ;) for me the worse is clearly to have to manually generate the parser > > Anyhow, is > "lib/eolian/eo_lexer.c: lib/eolian/eo_lexer.rl" > > Really needed? The whole point of adding the .rl.c rule, is that it's > done automatically, no? you're damn right ! > > -- > Tom. > > On 10/03/14 17:00, Jérémy Zurcher wrote: > > jeyzu pushed a commit to branch master. > > > > http://git.enlightenment.org/core/efl.git/commit/?id=995eac166a22b61054be0ccc8901db04f49a1ec7 > > > > commit 995eac166a22b61054be0ccc8901db04f49a1ec7 > > Author: Jérémy Zurcher <jer...@asynk.ch> > > Date: Mon Mar 10 18:14:16 2014 +0100 > > > > eolian: generate eo_lexer.c with ragel if available > > --- > > configure.ac | 5 +++++ > > src/Makefile_Eolian.am | 13 +++++++++++++ > > 2 files changed, 18 insertions(+) > > > > diff --git a/configure.ac b/configure.ac > > index 0d7e464..0b0dac3 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -330,6 +330,11 @@ fi > > > > AM_CONDITIONAL([EFL_ENABLE_COVERAGE], [test "${want_coverage}" = "yes"]) > > > > +# ragel > > + > > +AC_CHECK_PROG([have_ragel], [ragel], [yes], [no]) > > +AM_CONDITIONAL([EFL_HAVE_RAGEL], [test "${have_ragel}" = "yes"]) > > + > > #### Checks for libraries > > > > # check unit testing library > > diff --git a/src/Makefile_Eolian.am b/src/Makefile_Eolian.am > > index 871ffff..6807557 100644 > > --- a/src/Makefile_Eolian.am > > +++ b/src/Makefile_Eolian.am > > @@ -42,3 +42,16 @@ bin_eolian_eolian_gen_DEPENDENCIES = > > @USE_EOLIAN_INTERNAL_LIBS@ > > include Makefile_Eolian_Helper.am > > > > EXTRA_DIST += lib/eolian/eo_lexer.rl > > + > > +if EFL_HAVE_RAGEL > > +SUFFIXES += .rl > > + > > +AM_V_RAGEL = $(am__v_RAGEL_@AM_V@) > > +am__v_RAGEL_ = $(am__v_RAGEL_@AM_DEFAULT_V@) > > +am__v_RAGEL_0 = @echo " RAGEL " $@; > > + > > +.rl.c: > > + $(AM_V_RAGEL) ragel -o $@ $< > > + > > +lib/eolian/eo_lexer.c: lib/eolian/eo_lexer.rl > > +endif > > > > > > > ------------------------------------------------------------------------------ > Learn Graph Databases - Download FREE O'Reilly Book > "Graph Databases" is the definitive new guide to graph databases and their > applications. Written by three acclaimed leaders in the field, > this first edition is now available. Download your free book today! > http://p.sf.net/sfu/13534_NeoTech > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel --- Hell'O from Yverdoom
Jérémy (jeyzu) ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. Written by three acclaimed leaders in the field, this first edition is now available. Download your free book today! http://p.sf.net/sfu/13534_NeoTech _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
