[PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Brian Cain]

The hexagon idef-parser requires flex and bison as host build
tools.  Add them to the emsdk-wasm64-cross container image so that
wasm64 builds including hexagon-softmmu can find them.

Link: https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
---
 tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
index 8a924816f92..5debbc77d08 100644
--- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
+++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
@@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
 ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
 RUN apt-get update && apt-get install -y \
     autoconf \
+    bison \
     build-essential \
+    flex \
     libglib2.0-dev \
     libtool \
     pkgconf \
-- 
2.34.1

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Philippe Mathieu-Daudé]

On 24/6/26 07:48, Brian Cain wrote:
> The hexagon idef-parser requires flex and bison as host build
> tools.  Add them to the emsdk-wasm64-cross container image so that
> wasm64 builds including hexagon-softmmu can find them.
> 
> Link: https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> ---
>   tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> index 8a924816f92..5debbc77d08 100644
> --- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> +++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> @@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
>   ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
>   RUN apt-get update && apt-get install -y \
>       autoconf \
> +    bison \
>       build-essential \
> +    flex \
>       libglib2.0-dev \
>       libtool \
>       pkgconf \

Maybe emsdk-wasm64 should have been added to lci-tool first.
(not your responsibility).

Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Stefan Hajnoczi]

On Wed, Jun 24, 2026 at 1:48 AM Brian Cain <brian.cain@oss.qualcomm.com> wrote:
>
> The hexagon idef-parser requires flex and bison as host build
> tools.  Add them to the emsdk-wasm64-cross container image so that
> wasm64 builds including hexagon-softmmu can find them.
>
> Link: https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> ---
>  tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> index 8a924816f92..5debbc77d08 100644
> --- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> +++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> @@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
>  ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
>  RUN apt-get update && apt-get install -y \
>      autoconf \
> +    bison \
>      build-essential \
> +    flex \
>      libglib2.0-dev \
>      libtool \
>      pkgconf \

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Pierrick Bouvier]

On 6/23/2026 10:48 PM, Brian Cain wrote:
> The hexagon idef-parser requires flex and bison as host build
> tools.  Add them to the emsdk-wasm64-cross container image so that
> wasm64 builds including hexagon-softmmu can find them.
> 
> Link: https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> ---
>  tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
>  1 file changed, 2 insertions(+)
> 

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Alex Bennée]

Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> writes:

> On 24/6/26 07:48, Brian Cain wrote:
>> The hexagon idef-parser requires flex and bison as host build
>> tools.  Add them to the emsdk-wasm64-cross container image so that
>> wasm64 builds including hexagon-softmmu can find them.
>> Link:
>> https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
>> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
>> ---
>>   tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
>>   1 file changed, 2 insertions(+)
>> diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
>> b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
>> index 8a924816f92..5debbc77d08 100644
>> --- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
>> +++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
>> @@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
>>   ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
>>   RUN apt-get update && apt-get install -y \
>>       autoconf \
>> +    bison \
>>       build-essential \
>> +    flex \
>>       libglib2.0-dev \
>>       libtool \
>>       pkgconf \
>
> Maybe emsdk-wasm64 should have been added to lci-tool first.
> (not your responsibility).

I don't think that will be easy on account of emsdk having to build a
bunch of libraries to support building QEMU. If a distro provides a
better cross environment for wasm64 then we could try.
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Daniel P. Berrangé]

On Wed, Jun 24, 2026 at 06:40:24PM +0100, Alex Bennée wrote:
> Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> writes:
> 
> > On 24/6/26 07:48, Brian Cain wrote:
> >> The hexagon idef-parser requires flex and bison as host build
> >> tools.  Add them to the emsdk-wasm64-cross container image so that
> >> wasm64 builds including hexagon-softmmu can find them.
> >> Link:
> >> https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
> >> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> >> ---
> >>   tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
> >>   1 file changed, 2 insertions(+)
> >> diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> >> b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> >> index 8a924816f92..5debbc77d08 100644
> >> --- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> >> +++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> >> @@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
> >>   ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
> >>   RUN apt-get update && apt-get install -y \
> >>       autoconf \
> >> +    bison \
> >>       build-essential \
> >> +    flex \
> >>       libglib2.0-dev \
> >>       libtool \
> >>       pkgconf \
> >
> > Maybe emsdk-wasm64 should have been added to lci-tool first.
> > (not your responsibility).
> 
> I don't think that will be easy on account of emsdk having to build a
> bunch of libraries to support building QEMU. If a distro provides a
> better cross environment for wasm64 then we could try.

Yeah, there's rather too much custom stuff going on in our emsdk
dockerfile to make it practical to add to lcitool.


With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|

Re: [PATCH] tests/docker: add flex and bison to emsdk-wasm64-cross

[Pierrick Bouvier]

On 6/23/2026 10:48 PM, Brian Cain wrote:
> The hexagon idef-parser requires flex and bison as host build
> tools.  Add them to the emsdk-wasm64-cross container image so that
> wasm64 builds including hexagon-softmmu can find them.
> 
> Link: https://lore.kernel.org/qemu-devel/CAJSP0QVk6wsOnPAspC4YfXRp90saKibQfD4Mk-44-RQo0k=z3w@mail.gmail.com/
> Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
> ---
>  tests/docker/dockerfiles/emsdk-wasm64-cross.docker | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> index 8a924816f92..5debbc77d08 100644
> --- a/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> +++ b/tests/docker/dockerfiles/emsdk-wasm64-cross.docker
> @@ -19,7 +19,9 @@ ENV CXXFLAGS="$CFLAGS"
>  ENV LDFLAGS="-sWASM_BIGINT -sASYNCIFY=1 -L$TARGET/lib"
>  RUN apt-get update && apt-get install -y \
>      autoconf \
> +    bison \
>      build-essential \
> +    flex \
>      libglib2.0-dev \
>      libtool \
>      pkgconf \

Feel free to pull this directly as part of an Hexagon PR.

Regards,
Pierrick