All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
To: Peter Maydell <peter.maydell@linaro.org>, Emmanuel Blot <eblot@meta.com>
Cc: "Daniel P. Berrangé" <berrange@redhat.com>,
	qemu-devel@nongnu.org, "Thomas Huth" <th.huth+qemu@posteo.eu>,
	"Philippe Mathieu-Daudé" <philmd@mailo.com>,
	"Fabiano Rosas" <farosas@suse.de>,
	"Laurent Vivier" <lvivier@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: [PATCH] tests: strip macOS "-unsigned" suffix from binary name
Date: Mon, 8 Jun 2026 09:43:51 -0700	[thread overview]
Message-ID: <87a61b17-7200-47cc-bdb5-e8763c6c010a@oss.qualcomm.com> (raw)
In-Reply-To: <CAFEAcA_WtRMz93Y7CrGW0PDTzoA+jXtY-4e0AgYXemCe4GG6RQ@mail.gmail.com>

On 6/8/2026 9:08 AM, Peter Maydell wrote:
> On Mon, 8 Jun 2026 at 16:57, Emmanuel Blot <eblot@meta.com> wrote:
>>
>> Ok thanks, got it.
>>
>> I need to use `ninja -C build qemy-system-arm` to generate the signed
>> binary, because the default target only generates the `-unsigned`
>> version (I guess this is the linker's default output, macOS requires
>> an extra signing step).
>> The signed version is defined as an on-demand command which is not
>> part of the default build (`all`).
> 
> Right, this is the bug that needs fixing. The default target should
> build the signed binary.
> 
> -- PMM
> 

For information, I recently offered to Alex to do a rewrite of tcg
makefiles in meson natively. It's not the first time, and certainly not
the last time, that we waste hours/days debugging dependencies there,
this has to stop.

After this weekend, I am close from a proof of concept covering aarch64
and multiarch tests. It fixes any issue we had with dependencies, among
other improvements.

So hopefully the fix for that will be to wait for the meson version. If
anyone wants to spend cycle and fix existing Makefiles, feel free to do
it also, but they will be gone at some point.

Regards,
Pierrick


  reply	other threads:[~2026-06-08 16:44 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-08 15:24 [PATCH] tests: strip macOS "-unsigned" suffix from binary name Emmanuel Blot
2026-06-08 15:29 ` Peter Maydell
2026-06-08 15:35   ` Daniel P. Berrangé
2026-06-08 15:38     ` Peter Maydell
2026-06-08 15:56       ` Emmanuel Blot
2026-06-08 16:08         ` Peter Maydell
2026-06-08 16:43           ` Pierrick Bouvier [this message]
2026-06-08 17:39             ` Peter Maydell

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=87a61b17-7200-47cc-bdb5-e8763c6c010a@oss.qualcomm.com \
    --to=pierrick.bouvier@oss.qualcomm.com \
    --cc=alex.bennee@linaro.org \
    --cc=berrange@redhat.com \
    --cc=eblot@meta.com \
    --cc=farosas@suse.de \
    --cc=lvivier@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@mailo.com \
    --cc=qemu-devel@nongnu.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.