From: "Daniel P. Berrangé" <berrange@redhat.com>
To: He Zhe <zhe.he@windriver.com>
Cc: pbonzini@redhat.com, thuth@redhat.com, philmd@redhat.com,
laurent@vivier.eu, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] configure: Add pkg-config handling for libgcrypt
Date: Thu, 29 Aug 2019 10:36:48 +0100 [thread overview]
Message-ID: <20190829093648.GC17141@redhat.com> (raw)
In-Reply-To: <7d3d9777-725c-7e27-515d-fc003351704f@windriver.com>
On Thu, Aug 29, 2019 at 05:26:49PM +0800, He Zhe wrote:
>
>
> On 8/29/19 5:15 PM, Daniel P. Berrangé wrote:
> > On Thu, Aug 29, 2019 at 04:53:02PM +0800, zhe.he@windriver.com wrote:
> >> From: He Zhe <zhe.he@windriver.com>
> >>
> >> libgcrypt may also be controlled by pkg-config, this patch adds pkg-config
> >> handling for libgcrypt.
> > Where are you seeing pkg-config files for libgcrypt ?
> >
> > The upstream project has (frustratingly) been hostile to any proposal to
> > add pkg-config support saying people should stick with their custom
> > libgcrypt-config tool
> >
> > https://dev.gnupg.org/T2037
> >
> > Even if this is something added by some distro downstream, what is the
> > benefit in using it, compared with libgcrypt-confg which should already
> > work & is portable.
>
> IMHO, it could be easy for people to use pkg-config as a center to control
> configurations for many different packages.
>
> This is just an addition for qemu to be able to work in both cases. It does not
> remove libgcrypt-confg and can fall back to libgcrypt-confg when pkg-config does
> not work.
The addition has a maint cost associated with it, since we have have two
different ways to achieve the same thing. When only one of the approaches
is provided by upstream, the other is not going to be widely tested. In
maintaining packages in Fedora which rely on pkg-config files that are
not upstream, we've seen frequent breakage when. So my preference is
stick with what we have that is supported by upstream gcrypt.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2019-08-29 9:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-29 8:53 [Qemu-devel] [PATCH] configure: Add pkg-config handling for libgcrypt zhe.he
2019-08-29 9:15 ` Daniel P. Berrangé
2019-08-29 9:26 ` He Zhe
2019-08-29 9:36 ` Daniel P. Berrangé [this message]
2022-01-07 11:43 ` Andrea Bolognani
2022-01-07 11:55 ` Thomas Huth
2022-01-07 12:06 ` Daniel P. Berrangé
2022-01-07 13:39 ` Andrea Bolognani
-- strict thread matches above, loose matches on Subject: below --
2019-08-29 8:51 zhe.he
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=20190829093648.GC17141@redhat.com \
--to=berrange@redhat.com \
--cc=laurent@vivier.eu \
--cc=pbonzini@redhat.com \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
--cc=zhe.he@windriver.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).