Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gpick will not compile -- gets stuck at source/cssparser/css_grammar_wrap.cpp #29

Open
thezbyg opened this issue Mar 21, 2015 · 4 comments

Comments

@thezbyg
Copy link
Owner

thezbyg commented Mar 21, 2015

Original issue 29 created by thezbyg on 2011-04-27T13:13:14.000Z:

  1. run 'scons' in the gpick hg checkout directory
  2. build continues okay until reaching css parsing.

flex --header-file=build/source/cssparser/css_lex.h -o build/source/cssparser/css_lex.cpp source/cssparser/css_lex.l
buildAction(["build/source/cssparser/css_grammar.c", "build/source/cssparser/css_grammar.h", "build/source/cssparser/css_grammar.c"], ["source/cssparser/css_grammar.y", "source/cssparser/css_grammar.y"])
Copy("build/lempar.c", "extern/lempar.c")
Copy("build/source/cssparser/css_grammar.y", "source/cssparser/css_grammar.y")
Delete("build/source/cssparser/css_grammar.y")
Compiling ==> build/source/cssparser/css_grammar_wrap.o
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c: In function ‘int yy_find_shift_action(yyParser*, unsigned char)’:
../build/source/cssparser/css_grammar.c:667:15: error: ‘YY_SHIFT_COUNT’ was not declared in this scope
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c:673:17: error: ‘YY_ACTTAB_COUNT’ was not declared in this scope
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c: In function ‘int yy_find_reduce_action(int, unsigned char)’:
../build/source/cssparser/css_grammar.c:731:15: error: ‘YY_REDUCE_COUNT’ was not declared in this scope
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c:742:17: error: ‘YY_ACTTAB_COUNT’ was not declared in this scope

I am using gcc 4.6, scons 2.0.1, flex 2.5.35 on Arch Linux.
I encountered this bug while adding a 'LAB' mixing mode to 'Mix Colors'. It occurs whether or not I have modified any code.

I tried copying in a more recent lempar.c from http://code.google.com/p/lemonbind/source/browse/lempar.c -- That reduces the errors but does not eliminate them

buildAction(["build/source/cssparser/css_grammar.c", "build/source/cssparser/css_grammar.h", "build/source/cssparser/css_grammar.c"], ["source/cssparser/css_grammar.y", "source/cssparser/css_grammar.y"])
Copy("build/lempar.c", "extern/lempar.c")
Copy("build/source/cssparser/css_grammar.y", "source/cssparser/css_grammar.y")
Delete("build/source/cssparser/css_grammar.y")
Compiling ==> build/source/cssparser/css_grammar_wrap.o
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c:101:26: error: uninitialized const ‘yyzerominor’ [-fpermissive]
../build/source/cssparser/css_grammar.c:70:15: note: ‘const union YYMINORTYPE’ has no user-provided default constructor
In file included from source/cssparser/css_grammar_wrap.cpp:20:0:
../build/source/cssparser/css_grammar.c: In function ‘void yy_reduce(yyParser_, int)’:
../build/source/cssparser/css_grammar.c:965:39: error: ‘yytestcase’ was not declared in this scope
scons: *_* [build/source/cssparser/css_grammar_wrap.o] Error 1

@thezbyg
Copy link
Owner Author

thezbyg commented Mar 21, 2015

Comment #1 originally posted by thezbyg on 2011-04-27T16:47:06.000Z:

Please post the contents of 'build/source/cssparser/css_grammar.c
' after the compilation fails (using original lempar.c file). I managed to successfully compile using lempar.c from http://code.google.com/p/lemonbind/source/browse/lempar.c, but it would cause segfaults when actually trying to use the generated parser. If you like to try it anyway, I attached the necessary changes.

@thezbyg
Copy link
Owner Author

thezbyg commented Mar 21, 2015

Comment #2 originally posted by thezbyg on 2011-04-28T00:10:20.000Z:

build/source/cssparser/css_grammar.c is attached (before the below mentioned changes)

Thanks for the patch! It didn't seem to apply (to lempar.c), then I copied the lines into build/source/cssparser/css_grammar.c and then it compiled ok, and I could use the LAB mixing mode :)

@thezbyg
Copy link
Owner Author

thezbyg commented Mar 21, 2015

Comment #3 originally posted by thezbyg on 2012-02-18T12:00:02.000Z:

What is the status of this?

I attach what, IME, is a working lempar.c.. it seems to contain changes that are the direct inverse of your patch. I realize I've had this sitting in my local clone of gpick for a while now.

@thezbyg
Copy link
Owner Author

thezbyg commented Mar 21, 2015

Comment #4 originally posted by thezbyg on 2012-02-18T12:06:48.000Z:

Never mind :|
Clearly this is not fixed. I don't really understand exactly what a fix for it would look like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant