On Mon, Oct 14, 2024 at 4:41 PM Peter Maydell <peter.mayd...@linaro.org> wrote:
>
> On Mon, 14 Oct 2024 at 15:38, Peter Maydell <peter.mayd...@linaro.org> wrote:
> >
> > On Mon, 14 Oct 2024 at 15:36, Paolo Bonzini <pbonz...@redhat.com> wrote:
> > >
> > > A working native Rust compiler is always needed in order to compile Rust
> > > code, even when cross compiling, in order to build the procedural macros
> > > that QEMU uses.
> > >
> > > Right now, the check is done in rust/qemu-api-macros/meson.build, but this
> > > has two disadvantages.  First, it makes the build fail when the Meson 
> > > "rust"
> > > option is set to "auto" (instead, Rust support should be disabled).  
> > > Second,
> > > add_languages() is one of the few functions that are executed even by
> > > "meson introspect", except that "meson introspect" executes both branches
> > > of "if" statements!  Therefore, "meson introspect" tries to look for a
> > > Rust compiler even if the option is disabled---and then fails because
> > > the compiler is required by rust/qemu-api-macros/meson.build.  This is
> > > visible for example if the compilation host has a stale
> > > scripts/meson-buildoptions.sh and no rustc installed.
> > >
> > > Both issues can be fixed by moving the check to the main meson.build,
> > > together with the check for the cross compiler.
> > >
> > > Reported-by: Peter Maydell <peter.mayd...@linaro.org>
> > > ---
> > >  meson.build                      | 3 ++-
> > >  rust/qemu-api-macros/meson.build | 2 --
> > >  2 files changed, 2 insertions(+), 3 deletions(-)
> >
> > Missing Signed-off-by: line ?

Ugh, yes (my OpenBSD VM does not have the usual git hooks :)).

Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>

> Oh, and is there any better order in which to apply this
> patch and your other fix, or does it not matter?

It's not super important but I guess this one could go first, since it
affects non-incremental builds (as in CI) as well.

Paolo


Reply via email to