* [PATCH] Hexagon (meson.build): define min bison version
@ 2023-02-07 14:52 Matheus Tavares Bernardino
2023-02-07 14:54 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 10+ messages in thread
From: Matheus Tavares Bernardino @ 2023-02-07 14:52 UTC (permalink / raw)
To: qemu-devel; +Cc: anjo, tsimpson, bcain, quic_acaggian
Hexagon's idef-parser machinery uses some bison features that are not
available at older versions. The most preeminent example (as it can
be used as a sentinel) is "%define parse.error verbose". This was
introduced in version 3.0 of the tool, which is able to compile
qemu-hexagon just fine. However, compilation fails with the previous
minor bison release, v2.7. So let's assert the minimum version at
meson.build to give a more comprehensive error message for those trying
to compile QEMU.
[1]: https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
---
target/hexagon/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
index c9d31d095c..42b03c81e6 100644
--- a/target/hexagon/meson.build
+++ b/target/hexagon/meson.build
@@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user' in target_dirs
)
bison = generator(
- find_program('bison'),
+ find_program('bison', version: '>=3.0'),
output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
arguments: ['@INPUT@', '--defines=@OUTPUT1@', '--output=@OUTPUT0@']
)
--
2.37.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 14:52 [PATCH] Hexagon (meson.build): define min bison version Matheus Tavares Bernardino
@ 2023-02-07 14:54 ` Philippe Mathieu-Daudé
2023-02-07 15:08 ` Thomas Huth
0 siblings, 1 reply; 10+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-02-07 14:54 UTC (permalink / raw)
To: Matheus Tavares Bernardino, qemu-devel
Cc: anjo, tsimpson, bcain, quic_acaggian, Thomas Huth,
Daniel P. Berrangé,
Paolo Bonzini
Cc'ing Paolo/Daniel/Thomas
On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
> Hexagon's idef-parser machinery uses some bison features that are not
> available at older versions. The most preeminent example (as it can
> be used as a sentinel) is "%define parse.error verbose". This was
> introduced in version 3.0 of the tool, which is able to compile
> qemu-hexagon just fine. However, compilation fails with the previous
> minor bison release, v2.7. So let's assert the minimum version at
> meson.build to give a more comprehensive error message for those trying
> to compile QEMU.
>
> [1]: https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
>
> Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
> ---
> target/hexagon/meson.build | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
> index c9d31d095c..42b03c81e6 100644
> --- a/target/hexagon/meson.build
> +++ b/target/hexagon/meson.build
> @@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user' in target_dirs
> )
>
> bison = generator(
> - find_program('bison'),
> + find_program('bison', version: '>=3.0'),
> output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
> arguments: ['@INPUT@', '--defines=@OUTPUT1@', '--output=@OUTPUT0@']
> )
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 14:54 ` Philippe Mathieu-Daudé
@ 2023-02-07 15:08 ` Thomas Huth
2023-02-07 15:49 ` Taylor Simpson
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Thomas Huth @ 2023-02-07 15:08 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Matheus Tavares Bernardino, qemu-devel
Cc: anjo, tsimpson, bcain, quic_acaggian, Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial
On 07/02/2023 15.54, Philippe Mathieu-Daudé wrote:
> Cc'ing Paolo/Daniel/Thomas
>
> On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
>> Hexagon's idef-parser machinery uses some bison features that are not
>> available at older versions. The most preeminent example (as it can
>> be used as a sentinel) is "%define parse.error verbose". This was
>> introduced in version 3.0 of the tool, which is able to compile
>> qemu-hexagon just fine. However, compilation fails with the previous
>> minor bison release, v2.7. So let's assert the minimum version at
>> meson.build to give a more comprehensive error message for those trying
>> to compile QEMU.
>>
>> [1]:
>> https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
>>
>>
>> Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
>> ---
>> target/hexagon/meson.build | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
>> index c9d31d095c..42b03c81e6 100644
>> --- a/target/hexagon/meson.build
>> +++ b/target/hexagon/meson.build
>> @@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user' in
>> target_dirs
>> )
>> bison = generator(
>> - find_program('bison'),
>> + find_program('bison', version: '>=3.0'),
>> output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
>> arguments: ['@INPUT@', '--defines=@OUTPUT1@', '--output=@OUTPUT0@']
>> )
Looks reasonable, thus:
Reviewed-by: Thomas Huth <thuth@redhat.com>
Out of curiosity: Where did you encounter this problem? After having a quick
look at https://repology.org/project/bison/versions it seems to me that all
our supported OS distros should already ship bison 3.0 or newer...
Thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 15:08 ` Thomas Huth
@ 2023-02-07 15:49 ` Taylor Simpson
2023-02-07 16:07 ` Alessandro Di Federico via
2023-02-07 16:11 ` Philippe Mathieu-Daudé
2023-02-07 16:56 ` Matheus Tavares Bernardino
2 siblings, 1 reply; 10+ messages in thread
From: Taylor Simpson @ 2023-02-07 15:49 UTC (permalink / raw)
To: Thomas Huth, Philippe Mathieu-Daudé,
Matheus Bernardino (QUIC),
qemu-devel
Cc: anjo, Brian Cain, Antonio Caggiano (QUIC),
Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial, Alessandro Di Federico
> -----Original Message-----
> From: Thomas Huth <thuth@redhat.com>
> Sent: Tuesday, February 7, 2023 9:08 AM
> To: Philippe Mathieu-Daudé <philmd@linaro.org>; Matheus Bernardino
> (QUIC) <quic_mathbern@quicinc.com>; qemu-devel@nongnu.org
> Cc: anjo@rev.ng; Taylor Simpson <tsimpson@quicinc.com>; Brian Cain
> <bcain@quicinc.com>; Antonio Caggiano (QUIC)
> <quic_acaggian@quicinc.com>; Daniel P. Berrangé <berrange@redhat.com>;
> Paolo Bonzini <pbonzini@redhat.com>; QEMU Trivial <qemu-
> trivial@nongnu.org>
> Subject: Re: [PATCH] Hexagon (meson.build): define min bison version
>
> On 07/02/2023 15.54, Philippe Mathieu-Daudé wrote:
> > Cc'ing Paolo/Daniel/Thomas
> >
> > On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
> >> Hexagon's idef-parser machinery uses some bison features that are not
> >> available at older versions. The most preeminent example (as it can
> >> be used as a sentinel) is "%define parse.error verbose". This was
> >> introduced in version 3.0 of the tool, which is able to compile
> >> qemu-hexagon just fine. However, compilation fails with the previous
> >> minor bison release, v2.7. So let's assert the minimum version at
> >> meson.build to give a more comprehensive error message for those
> >> trying to compile QEMU.
> >>
> >> [1]:
> >> https://www.gnu.org/software/bison/manual/html_node/_0025define-
> Summa
> >> ry.html#index-_0025define-parse_002eerror
> >>
> >>
> >> Signed-off-by: Matheus Tavares Bernardino
> <quic_mathbern@quicinc.com>
> >> ---
> >> target/hexagon/meson.build | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
> >> index c9d31d095c..42b03c81e6 100644
> >> --- a/target/hexagon/meson.build
> >> +++ b/target/hexagon/meson.build
> >> @@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user'
> >> in target_dirs
> >> )
> >> bison = generator(
> >> - find_program('bison'),
> >> + find_program('bison', version: '>=3.0'),
> >> output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
> >> arguments: ['@INPUT@', '--defines=@OUTPUT1@', '--
> output=@OUTPUT0@']
> >> )
>
> Looks reasonable, thus:
>
> Reviewed-by: Thomas Huth <thuth@redhat.com>
>
> Out of curiosity: Where did you encounter this problem? After having a quick
> look at https://repology.org/project/bison/versions it seems to me that all
> our supported OS distros should already ship bison 3.0 or newer...
>
> Thomas
CC'ing Alessandro
Reviewed-by: Taylor Simpson <tsimpson@quicinc.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 15:49 ` Taylor Simpson
@ 2023-02-07 16:07 ` Alessandro Di Federico via
0 siblings, 0 replies; 10+ messages in thread
From: Alessandro Di Federico via @ 2023-02-07 16:07 UTC (permalink / raw)
To: Taylor Simpson
Cc: Thomas Huth, Philippe Mathieu-Daudé,
Matheus Bernardino (QUIC),
qemu-devel, anjo, Brian Cain, Antonio Caggiano (QUIC),
Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial
The patch makes sense to me.
Reviewed-by: Alessandro Di Federico <ale@rev.ng>
--
Alessandro Di Federico
rev.ng Labs
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 15:08 ` Thomas Huth
2023-02-07 15:49 ` Taylor Simpson
@ 2023-02-07 16:11 ` Philippe Mathieu-Daudé
2023-02-07 16:12 ` Philippe Mathieu-Daudé
2023-02-08 8:53 ` Bastian Koppelmann
2023-02-07 16:56 ` Matheus Tavares Bernardino
2 siblings, 2 replies; 10+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-02-07 16:11 UTC (permalink / raw)
To: Thomas Huth, Matheus Tavares Bernardino, qemu-devel
Cc: anjo, tsimpson, bcain, quic_acaggian, Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial, Bastian Koppelmann
+Bastian
On 7/2/23 16:08, Thomas Huth wrote:
> On 07/02/2023 15.54, Philippe Mathieu-Daudé wrote:
>> Cc'ing Paolo/Daniel/Thomas
>>
>> On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
>>> Hexagon's idef-parser machinery uses some bison features that are not
>>> available at older versions. The most preeminent example (as it can
>>> be used as a sentinel) is "%define parse.error verbose". This was
>>> introduced in version 3.0 of the tool, which is able to compile
>>> qemu-hexagon just fine. However, compilation fails with the previous
>>> minor bison release, v2.7. So let's assert the minimum version at
>>> meson.build to give a more comprehensive error message for those trying
>>> to compile QEMU.
>>>
>>> [1]:
>>> https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
>>>
>>> Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
>>> ---
>>> target/hexagon/meson.build | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
>>> index c9d31d095c..42b03c81e6 100644
>>> --- a/target/hexagon/meson.build
>>> +++ b/target/hexagon/meson.build
>>> @@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user'
>>> in target_dirs
>>> )
>>> bison = generator(
>>> - find_program('bison'),
>>> + find_program('bison', version: '>=3.0'),
>>> output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
>>> arguments: ['@INPUT@', '--defines=@OUTPUT1@',
>>> '--output=@OUTPUT0@']
>>> )
>
> Looks reasonable, thus:
>
> Reviewed-by: Thomas Huth <thuth@redhat.com>
>
> Out of curiosity: Where did you encounter this problem? After having a
> quick look at https://repology.org/project/bison/versions it seems to me
> that all our supported OS distros should already ship bison 3.0 or newer...
I suppose this fix the tricore container problem Peter reported last week:
https://lore.kernel.org/qemu-devel/CAFEAcA-Vr8=br=9jGU1Tr=HTyH6o+S9H79oG=6BqZb8FSp+2Tw@mail.gmail.com/
https://gitlab.com/qemu-project/qemu/-/jobs/3710561054
#6 43.63 ldlex.l: In function 'yy_input':
#6 43.63 ldlex.l:615:7: error: 'yy_current_buffer' undeclared (first
use in this function); did you mean 'yy_create_buffer'?
#6 43.63 {
#6 43.63 ^
#6 43.63 yy_create_buffer
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 16:11 ` Philippe Mathieu-Daudé
@ 2023-02-07 16:12 ` Philippe Mathieu-Daudé
2023-02-08 8:53 ` Bastian Koppelmann
1 sibling, 0 replies; 10+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-02-07 16:12 UTC (permalink / raw)
To: Thomas Huth, Matheus Tavares Bernardino, qemu-devel
Cc: anjo, tsimpson, bcain, quic_acaggian, Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial, Bastian Koppelmann
On 7/2/23 17:11, Philippe Mathieu-Daudé wrote:
> +Bastian
>
> On 7/2/23 16:08, Thomas Huth wrote:
>> On 07/02/2023 15.54, Philippe Mathieu-Daudé wrote:
>>> Cc'ing Paolo/Daniel/Thomas
>>>
>>> On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
>>>> Hexagon's idef-parser machinery uses some bison features that are not
>>>> available at older versions. The most preeminent example (as it can
>>>> be used as a sentinel) is "%define parse.error verbose". This was
>>>> introduced in version 3.0 of the tool, which is able to compile
>>>> qemu-hexagon just fine. However, compilation fails with the previous
>>>> minor bison release, v2.7. So let's assert the minimum version at
>>>> meson.build to give a more comprehensive error message for those trying
>>>> to compile QEMU.
>>>>
>>>> [1]:
>>>> https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
>>>>
>>>> Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
>>>> ---
>>>> target/hexagon/meson.build | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
>>>> index c9d31d095c..42b03c81e6 100644
>>>> --- a/target/hexagon/meson.build
>>>> +++ b/target/hexagon/meson.build
>>>> @@ -183,7 +183,7 @@ if idef_parser_enabled and 'hexagon-linux-user'
>>>> in target_dirs
>>>> )
>>>> bison = generator(
>>>> - find_program('bison'),
>>>> + find_program('bison', version: '>=3.0'),
>>>> output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
>>>> arguments: ['@INPUT@', '--defines=@OUTPUT1@',
>>>> '--output=@OUTPUT0@']
>>>> )
>>
>> Looks reasonable, thus:
>>
>> Reviewed-by: Thomas Huth <thuth@redhat.com>
>>
>> Out of curiosity: Where did you encounter this problem? After having a
>> quick look at https://repology.org/project/bison/versions it seems to
>> me that all our supported OS distros should already ship bison 3.0 or
>> newer...
>
> I suppose this fix the tricore container problem Peter reported last week:
s/fix/fixes/
> https://lore.kernel.org/qemu-devel/CAFEAcA-Vr8=br=9jGU1Tr=HTyH6o+S9H79oG=6BqZb8FSp+2Tw@mail.gmail.com/
>
> https://gitlab.com/qemu-project/qemu/-/jobs/3710561054
>
> #6 43.63 ldlex.l: In function 'yy_input':
> #6 43.63 ldlex.l:615:7: error: 'yy_current_buffer' undeclared (first
> use in this function); did you mean 'yy_create_buffer'?
> #6 43.63 {
> #6 43.63 ^
> #6 43.63 yy_create_buffer
(if so, it would be helpful to have that mentioned in the commit
description)
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 15:08 ` Thomas Huth
2023-02-07 15:49 ` Taylor Simpson
2023-02-07 16:11 ` Philippe Mathieu-Daudé
@ 2023-02-07 16:56 ` Matheus Tavares Bernardino
2023-02-07 17:12 ` Daniel P. Berrangé
2 siblings, 1 reply; 10+ messages in thread
From: Matheus Tavares Bernardino @ 2023-02-07 16:56 UTC (permalink / raw)
To: thuth
Cc: anjo, bcain, berrange, pbonzini, philmd, qemu-devel,
qemu-trivial, quic_acaggian, quic_mathbern, tsimpson
Thomas Huth <thuth@redhat.com> wrote:
>
> Out of curiosity: Where did you encounter this problem? After having a quick
> look at https://repology.org/project/bison/versions it seems to me that all
> our supported OS distros should already ship bison 3.0 or newer...
I actually noticed this when compiling our downstream
qemu-system-hexagon on macOS (Ventura 13.2), where the bundled bison
version is 2.3 (although 3.8.2 is available through brew). I thought
this could affect other upstream users too, but good to know that the
supported OSes already ship bison >= 3.0.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 16:56 ` Matheus Tavares Bernardino
@ 2023-02-07 17:12 ` Daniel P. Berrangé
0 siblings, 0 replies; 10+ messages in thread
From: Daniel P. Berrangé @ 2023-02-07 17:12 UTC (permalink / raw)
To: Matheus Tavares Bernardino
Cc: thuth, anjo, bcain, pbonzini, philmd, qemu-devel, qemu-trivial,
quic_acaggian, tsimpson
On Tue, Feb 07, 2023 at 01:56:03PM -0300, Matheus Tavares Bernardino wrote:
> Thomas Huth <thuth@redhat.com> wrote:
> >
> > Out of curiosity: Where did you encounter this problem? After having a quick
> > look at https://repology.org/project/bison/versions it seems to me that all
> > our supported OS distros should already ship bison 3.0 or newer...
>
> I actually noticed this when compiling our downstream
> qemu-system-hexagon on macOS (Ventura 13.2), where the bundled bison
> version is 2.3 (although 3.8.2 is available through brew). I thought
> this could affect other upstream users too, but good to know that the
> supported OSes already ship bison >= 3.0.
FWIW, our testing on macOS heavily relies on the versions from brew.
You can see the list of brew pkgs we consume in
qemu.git/.gitlab-ci.d/cirrus/macos-12.vars
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Hexagon (meson.build): define min bison version
2023-02-07 16:11 ` Philippe Mathieu-Daudé
2023-02-07 16:12 ` Philippe Mathieu-Daudé
@ 2023-02-08 8:53 ` Bastian Koppelmann
1 sibling, 0 replies; 10+ messages in thread
From: Bastian Koppelmann @ 2023-02-08 8:53 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: Thomas Huth, Matheus Tavares Bernardino, qemu-devel, anjo,
tsimpson, bcain, quic_acaggian, Daniel P. Berrangé,
Paolo Bonzini, QEMU Trivial
Hi Phil,
On Tue, Feb 07, 2023 at 05:11:10PM +0100, Philippe Mathieu-Daudé wrote:
> +Bastian
>
> On 7/2/23 16:08, Thomas Huth wrote:
> > On 07/02/2023 15.54, Philippe Mathieu-Daudé wrote:
> > > Cc'ing Paolo/Daniel/Thomas
> > >
> > > On 7/2/23 15:52, Matheus Tavares Bernardino wrote:
> > > > Hexagon's idef-parser machinery uses some bison features that are not
> > > > available at older versions. The most preeminent example (as it can
> > > > be used as a sentinel) is "%define parse.error verbose". This was
> > > > introduced in version 3.0 of the tool, which is able to compile
> > > > qemu-hexagon just fine. However, compilation fails with the previous
> > > > minor bison release, v2.7. So let's assert the minimum version at
> > > > meson.build to give a more comprehensive error message for those trying
> > > > to compile QEMU.
> > > >
> > > > [1]: https://www.gnu.org/software/bison/manual/html_node/_0025define-Summary.html#index-_0025define-parse_002eerror
> > > >
> > > > Signed-off-by: Matheus Tavares Bernardino <quic_mathbern@quicinc.com>
> > > > ---
> > > > target/hexagon/meson.build | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
> > > > index c9d31d095c..42b03c81e6 100644
> > > > --- a/target/hexagon/meson.build
> > > > +++ b/target/hexagon/meson.build
> > > > @@ -183,7 +183,7 @@ if idef_parser_enabled and
> > > > 'hexagon-linux-user' in target_dirs
> > > > )
> > > > bison = generator(
> > > > - find_program('bison'),
> > > > + find_program('bison', version: '>=3.0'),
> > > > output: ['@BASENAME@.tab.c', '@BASENAME@.tab.h'],
> > > > arguments: ['@INPUT@', '--defines=@OUTPUT1@',
> > > > '--output=@OUTPUT0@']
> > > > )
> >
> > Looks reasonable, thus:
> >
> > Reviewed-by: Thomas Huth <thuth@redhat.com>
> >
> > Out of curiosity: Where did you encounter this problem? After having a
> > quick look at https://repology.org/project/bison/versions it seems to me
> > that all our supported OS distros should already ship bison 3.0 or
> > newer...
>
> I suppose this fix the tricore container problem Peter reported last week:
> https://lore.kernel.org/qemu-devel/CAFEAcA-Vr8=br=9jGU1Tr=HTyH6o+S9H79oG=6BqZb8FSp+2Tw@mail.gmail.com/
>
> https://gitlab.com/qemu-project/qemu/-/jobs/3710561054
>
> #6 43.63 ldlex.l: In function 'yy_input':
> #6 43.63 ldlex.l:615:7: error: 'yy_current_buffer' undeclared (first
> use in this function); did you mean 'yy_create_buffer'?
> #6 43.63 {
> #6 43.63 ^
> #6 43.63 yy_create_buffer
Thanks, I think this would fix it. However, I want to include tricore-gcc/newlib to the
docker image and created a toolchain release for that, so that we don't have to
compile binutils/gcc/newlib ourself. So I don't think this is relevant for me
anymore.
Cheers,
Bastian
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-02-08 8:54 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 14:52 [PATCH] Hexagon (meson.build): define min bison version Matheus Tavares Bernardino
2023-02-07 14:54 ` Philippe Mathieu-Daudé
2023-02-07 15:08 ` Thomas Huth
2023-02-07 15:49 ` Taylor Simpson
2023-02-07 16:07 ` Alessandro Di Federico via
2023-02-07 16:11 ` Philippe Mathieu-Daudé
2023-02-07 16:12 ` Philippe Mathieu-Daudé
2023-02-08 8:53 ` Bastian Koppelmann
2023-02-07 16:56 ` Matheus Tavares Bernardino
2023-02-07 17:12 ` Daniel P. Berrangé
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.