bug-bison
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: api.location.type is unsupported for C parsers


From: Scheidler , Balázs
Subject: Re: api.location.type is unsupported for C parsers
Date: Thu, 25 Apr 2019 07:03:38 +0200

I've tried this now and api.location.type does seem to work. I still have
some other compilation issues to hunt. One of those is on  "enum
XXX_tokentype". Since I am using the same tokens in multiple grammars, I
get duplications.

modules/native/native-grammar.c:279:25: error: expected identifier before
numeric constant
 #define LL_CONTEXT_ROOT 1
                         ^
lib/cfg-grammar.h:107:5: note: in expansion of macro ‘LL_CONTEXT_ROOT’
     LL_CONTEXT_ROOT = 1,
     ^~~~~~~~~~~~~~~

The native-grammar file is a "slave" grammar that reuses some of the tokens
of the main one. cfg-grammar.h is the main grammar. At the end both
grammars have overlapping "tokens", but then the 2nd includes the header of
the first. I am not sure how to resolve this yet, but I'll look at it in
the coming days.

I got to run, but I wanted to drop this note anyway. api.location,type
seems to work all-right, but still some issues to fix. :)

Thanks
Bazsi

On Tue, Apr 23, 2019 at 8:20 PM Scheidler, Balázs <
address@hidden> wrote:

>
>
> On Tue, Apr 23, 2019, 18:01 Akim Demaille <address@hidden wrote:
>
>> Hi Balázs
>>
>> > Le 23 avr. 2019 à 17:37, Scheidler, Balázs <
>> address@hidden> a écrit :
>> >
>> > The tarball didn't work for me, it may have been generated from a
>> different
>> > commit as the patch wasn't included.
>>
>> Ouch.  I apologize for this.  I took the last one in the list,
>> but the last one is not the latest :(
>>
>> > I've tried to bootstrap from git, but that gave me an error message.
>> > Autoconf submodule registration fails for some reason.
>>
>> You probably did not run `git submodule update --init`.
>>
>
> I did try that, and even --update but for some reason autoconf wasnt
> pulled whereas gnulib was.
>
>
>
>
>> > At this point i gave up. Can you generate a tarball that includes the
>> patch?
>>
>> Sure.  I'm sorry for the waste of time :(
>>
>
> Np.  Wasnt too long.
>
>>
>> https://www.lrde.epita.fr/~akim/private/bison/bison-3.3.2.138-67bc.tar.
>> <https://www.lrde.epita.fr/~akim/private/bison/bison-3.3.2.138-67bc.tar.xz>
>> gz
>>
>
> Thanks, will try again.
>
>
> Cheers!
>
>


reply via email to

[Prev in Thread] Current Thread [Next in Thread]