All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
To: "Alex Bennée" <alex.bennee@linaro.org>, qemu-devel@nongnu.org
Cc: "Brad Smith" <brad@comstyle.com>,
	"Jiaxun Yang" <jiaxun.yang@flygoat.com>,
	"Song Gao" <gaosong@loongson.cn>, "Warner Losh" <imp@bsdimp.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Cornelia Huck" <cohuck@redhat.com>,
	"Fabiano Rosas" <farosas@suse.de>, "John Snow" <jsnow@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@mailo.com>,
	"Thomas Huth" <th.huth+qemu@posteo.eu>,
	"Brian Cain" <brian.cain@oss.qualcomm.com>,
	qemu-arm@nongnu.org,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Kyle Evans" <kevans@freebsd.org>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Cleber Rosa" <crosa@redhat.com>
Subject: Re: [PATCH 1/9] scripts/meson-buildoptions.sh: update missing options
Date: Sun, 13 Sep 2026 22:00:02 -0700	[thread overview]
Message-ID: <d925904e-6a15-4c58-8fd0-e54d5724c364@oss.qualcomm.com> (raw)
In-Reply-To: <20260910181504.1010916-2-alex.bennee@linaro.org>

On 9/10/26 11:14 AM, Alex Bennée wrote:
> This should have been updated when the meson check-tcg changes where
> made. Include it now.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   scripts/meson-buildoptions.sh | 142 ++++++++++++++++++++++++++++++++++
>   1 file changed, 142 insertions(+)
> 

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

Thanks for this fix.
I can confirm I didn't see this needed to be updated, simply because I 
only ran make check-tcg (which didn't retrigger it), and use ninja/meson 
test for everything else.

Maybe the best place to generate this file would be from meson itself, 
as a custom_target run by default.

What do concerned folks think?

Regards,
Pierrick


  parent reply	other threads:[~2026-09-14  5:00 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-10 18:14 [PATCH 0/9] testing: various docker clean-ups post meson tcg conversion Alex Bennée
2026-09-10 18:14 ` [PATCH 1/9] scripts/meson-buildoptions.sh: update missing options Alex Bennée
2026-09-10 20:03   ` Fabiano Rosas
2026-09-10 22:17     ` Alex Bennée
2026-09-14  5:00   ` Pierrick Bouvier [this message]
2026-09-14 10:22   ` Paolo Bonzini
2026-09-10 18:14 ` [PATCH 2/9] tests/tcg: improve check for working cross compilers Alex Bennée
2026-09-14  5:08   ` Pierrick Bouvier
2026-09-14  8:34     ` Daniel P. Berrangé
2026-09-14 16:05       ` Pierrick Bouvier
2026-09-14 16:07       ` Alex Bennée
2026-09-14 23:55         ` Ilya Leoshkevich
2026-09-15  0:21           ` Pierrick Bouvier
2026-09-10 18:14 ` [PATCH 3/9] tests/tcg: prefix meson built docker tags with qemu/ Alex Bennée
2026-09-14  5:10   ` Pierrick Bouvier
2026-09-10 18:14 ` [PATCH 4/9] tests/tcg: use --add-current-user when building containers Alex Bennée
2026-09-14  5:15   ` Pierrick Bouvier
2026-09-14  7:27     ` Alex Bennée
2026-09-14 15:59       ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 5/9] tests/docker: add container registry to configure and use Alex Bennée
2026-09-14  5:21   ` Pierrick Bouvier
2026-09-14  7:28     ` Alex Bennée
2026-09-14 16:15       ` Pierrick Bouvier
2026-09-14 16:18       ` Pierrick Bouvier
2026-09-15 17:10         ` Alex Bennée
2026-09-15 17:49           ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 6/9] tests/docker: remove used _read_qemu_dockerfile Alex Bennée
2026-09-14  5:21   ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 7/9] tests/docker: strip remaining caching logic from docker.py Alex Bennée
2026-09-14  5:22   ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 8/9] tests/docker: add specific command to parser Alex Bennée
2026-09-14  5:22   ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 9/9] tests/docker: swap --quiet for --verbose Alex Bennée
2026-09-12  8:46 ` [PATCH 0/9] testing: various docker clean-ups post meson tcg conversion Richard Henderson
2026-09-12 17:24   ` Alex Bennée
2026-09-12 10:52 ` Richard Henderson
2026-09-12 19:37   ` Alex Bennée

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d925904e-6a15-4c58-8fd0-e54d5724c364@oss.qualcomm.com \
    --to=pierrick.bouvier@oss.qualcomm.com \
    --cc=alex.bennee@linaro.org \
    --cc=berrange@redhat.com \
    --cc=brad@comstyle.com \
    --cc=brian.cain@oss.qualcomm.com \
    --cc=cohuck@redhat.com \
    --cc=crosa@redhat.com \
    --cc=farosas@suse.de \
    --cc=gaosong@loongson.cn \
    --cc=imp@bsdimp.com \
    --cc=jiaxun.yang@flygoat.com \
    --cc=jsnow@redhat.com \
    --cc=kevans@freebsd.org \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@mailo.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=th.huth+qemu@posteo.eu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
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.