All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Alex Bennée" <alex.bennee@linaro.org>,
	"QEMU Developers" <qemu-devel@nongnu.org>
Subject: Re: [PULL 29/86] meson: move libudev test
Date: Thu, 1 Oct 2020 21:15:58 +0200	[thread overview]
Message-ID: <85451b4a-c07a-cf42-c058-e12c93a68b43@redhat.com> (raw)
In-Reply-To: <CAFEAcA9iGZBKAyDnZmb=ieuSZvb2FkkzFqE_Ax-5TNHjY8oy5w@mail.gmail.com>

On 01/10/20 20:51, Peter Maydell wrote:
> Yes, I have a lot of --disable-foo options. Ideally I wouldn't
> need any, because our configure/build system would identify "there
> isn't actually a static version of this dependency present"
> rather than blindly trusting pkg-config when it lies to us.
> (Is it possible to get Meson to just always do a "test that
> a trivial program with these cflags and libs will build" as
> part of whatever magic it does as part of dependency() ?

Unfortunately there are many special cases, including libraries that
require symbols in the executable, so its not possible to do that.  For
this reason the "static library not found for dependency" is just a warning.

We can add such a compile test ourselves.  Of course it might add to the
compile time if we do it for every dependency, but perhaps we can do one
with all the dependencies and if it fails loop on single dependencies to
give a better error.

Paolo



  reply	other threads:[~2020-10-01 19:18 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-30 19:58 [PULL v8 00/86] Misc QEMU patches for 2020-09-24 Paolo Bonzini
2020-09-30 19:58 ` [PULL 29/86] meson: move libudev test Paolo Bonzini
2020-10-01 16:19   ` Peter Maydell
2020-10-01 16:55     ` Paolo Bonzini
2020-10-01 18:51       ` Peter Maydell
2020-10-01 19:15         ` Paolo Bonzini [this message]
2020-09-30 19:58 ` [PULL 30/86] meson: move libmpathpersist test Paolo Bonzini
2020-09-30 19:58 ` [PULL 31/86] meson: extend libmpathpersist test for static linking Paolo Bonzini
2020-10-01 14:28 ` [PULL v8 00/86] Misc QEMU patches for 2020-09-24 Peter Maydell
2020-10-01 15:41   ` 罗勇刚(Yonggang Luo)
2020-10-02 15:58 ` Michal Prívozník
2020-10-02 16:22   ` Eduardo Habkost
2020-10-02 17:26     ` Michal Prívozník
2020-10-02 17:30       ` Paolo Bonzini
2020-10-02 17:55         ` Eduardo Habkost
2020-10-02 16:27   ` Paolo Bonzini
2020-10-02 17:01     ` Eduardo Habkost
2020-10-02 18:24     ` Eduardo Habkost
2020-10-02 18:45       ` Paolo Bonzini
2020-10-02 18:52         ` Peter Maydell
2020-10-07 10:57       ` Michal Privoznik

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=85451b4a-c07a-cf42-c058-e12c93a68b43@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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.