lua.js icon indicating copy to clipboard operation
lua.js copied to clipboard

Conflicts encountered

Open ggcrunchy opened this issue 12 years ago • 5 comments

Hi. I've been trying to build the new parser (the prebuilt version is now out-of-date), and I'm running into a LOT of "Resolve R/R conflict (use first production declared in grammar.) (2,21, 2,14) -> 2,14Resolve R/R conflict (use first production declared in grammar.)" sort of warnings.

These seem to be manifesting as errors along the lines of:

error during main.lua : Error: Parse error on line 26: ...s(tiles) do TILES[#TILES + 1] = love. ----------------------^ Expecting '=', ',', got '[' :

Rearranging some lines will give me similar errors with other operators.

(From https://dl.dropbox.com/u/14510154/TESTS/take_the_fork/main.lua )

I tried to tease out any parser shenanigans but found myself rather swamped. :smile:

I don't have much of an environment set up to run the makefile (I have Cygwin on some other machine...), but I ran it both through node and the web generator at http://zaach.github.com/jison/try/ with similar results.

Thanks in advance for any help or hints on this.

ggcrunchy avatar Nov 20 '12 07:11 ggcrunchy

I'm sorry I didn't get to this sooner, are you still experiencing these problems? I am unable to reproduce them.

mherkender avatar Dec 01 '12 00:12 mherkender

No worries. I did finally install Cygwin on this machine and try the actual makefile (versus just feeding the lua.jison into something and manually hooking stuff up). The relevant error now seems to be:

cd . && node src/build_lua_parser.js

C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:109 throw new Error(str); ^ Error: Lexical error on line 656. Unrecognized text. var indentLevel -------------------^ at Object.parseError (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:109:11) at Object.parseError (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:224:28) at Object.lexer.next (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:360:25) at Object.lex (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:365:22) at lex (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:131:28) at Object.parse (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\bnf-parser.js:144:26) at Object.parse (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\bnf.js:4:54) at Jison_Generator (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison.js:75:42) at (anonymous function) (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\typal.js:23:28) at new construct.o.constructor (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison\util\typal.js:77:70) at new Jison_Generator (C:\users\steve\desktop\lua.js\node_modules\jison\lib\jison.js:1571:20) Makefile:25: recipe for target `src/lua_parser.js' failed

(Seems to be around here: https://github.com/zaach/jison/blob/master/lib/jison/util/bnf-parser.js#L360)

This is a fresh Cygwin install with just 'devel', a fresh node install, and Windows Vista. Haven't tried on anything else.

ggcrunchy avatar Dec 01 '12 00:12 ggcrunchy

It's crashing when trying to build lua.jison. I'm not able to reproduce this, and while there are grammar problems, they're only warnings and it should work.

I've cleaned up the Makefile a bit, could you update and run make clean_all all? This will clear out basically everything that has been downloaded or generated, and build from scratch, allowing me to better understand what's going on.

mherkender avatar Dec 01 '12 01:12 mherkender

Sorry for the belated response. I've probably set this project aside until the new year, now that the contest is over.

That said, the make clean_all all gives me the same results. Some searching unfortunately seems to imply that obscure things can trigger this, e.g. this :frowning: And maybe it's some Windows arcana... (On that note, the pre-built version hasn't been updated.)

As an aside, have you seen this one (relatively new)? Maybe you have, but it wasn't in your stars.

ggcrunchy avatar Dec 14 '12 00:12 ggcrunchy

Odd. I'll take another look at it later. The prebuilt version has been updated, is it not working for you?

I have not heard of brozula, it's definitely interesting and reminds me of emscripten. Thanks for the link! It's good to see other people are exploring similar techniques and expanding Lua's range of influence.

mherkender avatar Dec 14 '12 04:12 mherkender